Bug #91649
closedLogin Failure Warning not sent when using PostgreSQL or MySQL strict mode
100%
Description
If a $GLOBALS['TYPO3_CONF_VARS']['BE']['warning_email_addr']
is set a DBAL-DriverException will occur in the backend log if there have been failed login attempts on a TYPO3 running with PostgreSQL:
Core: Exception handler (WEB): Uncaught TYPO3 Exception: An exception occurred while executing 'SELECT COUNT("uid") FROM "sys_log" WHERE ("type" = ?) AND ("action" = ?) AND ("error" <> ?) AND ("tstamp" > ?) ORDER BY "tstamp" ASC' with params [255, 3, 0, 1592228590]: SQLSTATE[42803]: Grouping error: 7 FEHLER: Spalte »sys_log.tstamp« muss in der GROUP-BY-Klausel erscheinen oder in einer Aggregatfunktion verwendet werden LINE 1: ... AND ("error" <> $3) AND ("tstamp" > $4) ORDER BY "tstamp" A... ^ Doctrine\DBAL\Exception\DriverException thrown in file /vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractPostgreSQLDriver.php in line 79. Requested URL: https://example.org/typo3/index.php?loginProvider=1433416747
This leads to the warning email not being sent.
Updated by Stephan Großberndt almost 4 years ago
- Project changed from TYPO3 Core to 1716
- Category deleted (
Security)
Updated by Markus Klein almost 4 years ago
So this is an invalid SQL query, which will also fail on MySQL strict mode (full_group_by)
Updated by Stephan Großberndt almost 4 years ago
- Project changed from 1716 to TYPO3 Core
- Subject changed from Login Failure Warning not sent when using PostgreSQL to Login Failure Warning not sent when using PostgreSQL or MySQL strict mode
- Category set to Authentication
- Complexity set to no-brainer
yes, exactly.
Updated by Gerrit Code Review almost 4 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/c/Packages/TYPO3.CMS/+/64899
Updated by Gerrit Code Review almost 4 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/c/Packages/TYPO3.CMS/+/64899
Updated by Gerrit Code Review over 3 years ago
Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65099
Updated by Gerrit Code Review over 3 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65100
Updated by Stephan Großberndt over 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 0ee172075b9589f5fcbd5004d8f2e40304c2cd7f.
Updated by Gerrit Code Review over 3 years ago
- Status changed from Resolved to Under Review
Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65100
Updated by Stephan Großberndt over 3 years ago
- Status changed from Under Review to Resolved
Applied in changeset 9b88e6bdba3a08e1806563471c68be8556733f63.
Updated by Gerrit Code Review over 3 years ago
- Status changed from Resolved to Under Review
Patch set 3 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65100
Updated by Stephan Großberndt over 3 years ago
- Status changed from Under Review to Resolved
Applied in changeset 5466944a16158a715c1dfafe848174323b500314.