Project

General

Profile

Actions

Task #100135

closed

Remove cookieWarning in ext:felogin

Added by Torben Hansen about 1 year ago. Updated 4 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2023-03-10
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The extension uses $GLOBALS['TSFE']->fe_user->isCookieSet() to determine, if a cookie warning message will be shown or not. The message is however never shown, since in a scenario, where the user login was successful, the isCookieSet() method will always return true, since AbstractUserAuthentication will return a cookie in the response to the client.

For TYPO3 12, the request token check will not even authenticate the user, if the pre-session cookie is not present. So ext:felogin will basically never know, if authentication was successful but cookies are disabled.

I could verify, that this feature is also broken in the legacy plugin of ext:felogin (tested TYPO3 8.7 - 10.4).

Since the current "cookie waring" in ext:felogin does not work and can not be fixed easily, it is best to remove it completely and rely on the default message shown, when the login was not successful. This message includes a hint, that authentication was not successful, either caused by wrong credentials or because of disabled cookies.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #99710: EXT:felogin - Show Cookie Warning doesn't workClosedTorben Hansen2023-01-25

Actions
Actions

Also available in: Atom PDF