Bug #32274

sessionTimeout for BE has does not work

Added by Simon Schaufelberger about 8 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2011-12-02
Due date:
% Done:

100%

TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Hello,
i setup the following config in localconf.php:

$TYPO3_CONF_VARS['BE']['sessionTimeout'] = '86400';

the problem is that after about 1 hour of inactivity i get the login screen so the setting that i will be logged in for 1 day does not work.

TYPO3 version: 4.5.

Associated revisions

Revision 719c1a2f (diff)
Added by Mario Rimann almost 8 years ago

[BUGFIX] sessionTimeout for BE does not work

Changes an is_string() check to a check out of t3lib_Utility_Math
which will also accept '86400' as a valid value and not only 86400.

Change-Id: I5a8bfe4b1015d9a6195a9831272bd7e5518110dc
Resolves: #32274
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/7079
Reviewed-by: Markus Klein
Reviewed-by: Simon Schaufelberger
Tested-by: Simon Schaufelberger
Reviewed-by: Michael Stucki
Reviewed-by: Oliver Klee
Tested-by: Markus Klein
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Revision db7fec27 (diff)
Added by Mario Rimann almost 8 years ago

[BUGFIX] sessionTimeout for BE does not work

Changes an is_string() check to a check out of t3lib_Utility_Math
which will also accept '86400' as a valid value and not only 86400.

Change-Id: Ia92c69c94cfe08db88d4cd4c92fa6f8c4320055c
Resolves: #32274
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/7353
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Revision 043f1c83 (diff)
Added by Mario Rimann almost 8 years ago

[BUGFIX] sessionTimeout for BE does not work

Changes an is_string() check to a check out of t3lib_Utility_Math
which will also accept '86400' as a valid value and not only 86400.

Change-Id: I469b532b67fc5ac523ced85616bcb869ec1e7b6d
Resolves: #32274
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/7354
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

History

#1 Updated by Markus Klein about 8 years ago

Hi Simon!

I guess the problem is this (t3lib_userauth, line 772):

    // A user was found
if (is_string($this->auth_timeout_field)) {
    $timeout = intval($user[$this->auth_timeout_field]); // Get timeout-time from usertable
} else {
    $timeout = intval($this->auth_timeout_field); // Get timeout from object
}

The check is with is_string().
Can you try if it works with the setting
$TYPO3_CONF_VARS['BE']['sessionTimeout'] = 86400;

Regards
Markus

#2 Updated by Michael Stucki about 8 years ago

  • Category set to Backend API
  • Status changed from New to Needs Feedback
  • Complexity set to easy

#3 Updated by Michael Stucki about 8 years ago

Sounds logical indeed. I suggest to replace the is_string check with t3lib_utility_Math::canBeInterpretedAsInteger().

#4 Updated by Gerrit Code Review about 8 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7079

#5 Updated by Gerrit Code Review about 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7079

#6 Updated by Gerrit Code Review about 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7079

#7 Updated by Mario Rimann almost 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#8 Updated by Gerrit Code Review almost 8 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/7353

#9 Updated by Gerrit Code Review almost 8 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/7354

#10 Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/7354

#11 Updated by Mario Rimann almost 8 years ago

  • Status changed from Under Review to Resolved

#12 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF