Bug #78695

FrontendUserAuthentication timeout can't be less then 6000s

Added by Alexander Opitz almost 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Frontend
Target version:
Start date:
2016-11-14
Due date:
% Done:

100%

TYPO3 Version:
3.5
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
On Location Sprint

Description

In FrontendUserAuthentication::start the $this->sessionTimeout will be set to $this->lifetime but only if $this->lifetime is higher then default of $this->sessionTimeout (which is 6000).

$this->lifetime gets filled in TypoScriptFrontendController::initFEuser() with the value of $GLOBALS['TYPO3_CONF_VARS']['FE']['lifetime'];

I don't think, that this behavior is desired.

Original issue:

Session timeout longer than industry best practice


Related issues

Related to TYPO3 Core - Bug #14971: cookie lifetime from the localconf will be ignored Closed 2005-09-15

Associated revisions

Revision 308b75fc (diff)
Added by Alexander Opitz almost 2 years ago

[FEATURE] Possibility to set sessionTimeout for Frontend Users

Currently it was only possible to set the session timeout for the backend users
You can define the sessionTimeout with the new configuration option
$GLOBALS['TYPO3_CONF_VARS']['FE']['sessionTimeout'].

Resolves: #78695
Releases: master
Change-Id: I628ccc4448230af6dd2bf1533777375ef7e5624d
Reviewed-on: https://review.typo3.org/50654
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Gerrit Code Review almost 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#2 Updated by Gerrit Code Review almost 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#3 Updated by Gerrit Code Review almost 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#4 Updated by Gerrit Code Review almost 3 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#5 Updated by Alexander Opitz almost 3 years ago

  • TYPO3 Version changed from 6.2 to 3.5

The commit https://github.com/TYPO3/TYPO3.CMS/commit/385cacdfa7c81a9249ea7e458755942f1b96accb claimed to fix https://forge.typo3.org/issues/14971 (Mantis 1454) which also change file comments incorrectly.
That's why my patch fail.

#6 Updated by Gerrit Code Review almost 3 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#7 Updated by Gerrit Code Review almost 3 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#8 Updated by Gerrit Code Review almost 3 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#9 Updated by Benni Mack over 2 years ago

  • Target version changed from 8 LTS to next-patchlevel

#10 Updated by Benni Mack almost 2 years ago

  • Sprint Focus set to On Location Sprint

#11 Updated by Gerrit Code Review almost 2 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#12 Updated by Gerrit Code Review almost 2 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#13 Updated by Gerrit Code Review almost 2 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50654

#14 Updated by Alexander Opitz almost 2 years ago

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

#15 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF