Bug #33311

AuthenticationProviderManager::isAuthenticated does not work in authentication request

Added by Kira Backes over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Must have
Category:
Security
Target version:
-
Start date:
2012-01-19
Due date:
% Done:

100%

Estimated time:
PHP Version:
Has patch:
No
Complexity:

Description

If the client does not have a session and the client authenticates, then a new session is created. The method isAuthenticated checks if the current session canBeResumed(), but that method again checks for the session in $_COOKIE. Since the session was just created, the isAuthenticated method never works in the authentication request. It only starts working in all subsequent requests, since the new session is then available in $_COOKIE.


Related issues

Is duplicate of TYPO3.Flow - Bug #31905: getInterceptedRequest is NULL but ACL matches correctly DENYResolved2011-11-18

Actions
#1

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

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

#2

Updated by Gerrit Code Review over 9 years ago

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

#3

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/9262

#4

Updated by Christian Müller over 9 years ago

  • Assignee set to Christian Müller
#5

Updated by Gerrit Code Review over 9 years ago

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

#6

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/9302

#7

Updated by Christian Müller over 9 years ago

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

Also available in: Atom PDF