Bug #67703
closedNo logging to sysLog in ExceptionHandler
100%
Description
The sysext/core/Classes/Error/AbstractExceptionHandler.php class has a writeLog method which should write new entries to the sys_log table. It seems this is broken.
Within the method there is a check for proper DB connection:
if (is_object($GLOBALS['TYPO3_DB']) && !empty($GLOBALS['TYPO3_DB']->link)) {
The second check for link will always fail as the link property of TYPO3\CMS\Core\Database\DatabaseConnection is protected.
Perhaps this was caused due to a visibility change?
This should be handled like in typo3/sysext/core/Classes/Error/ErrorHandler.php
if (is_object($GLOBALS['TYPO3_DB']) && $GLOBALS['TYPO3_DB']->isConnected()) {
Tested in TYPO3 CMS 6.2.12 and 7.3
Updated by Gerrit Code Review almost 9 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 http://review.typo3.org/40565
Updated by Gerrit Code Review almost 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40565
Updated by Gerrit Code Review almost 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40565
Updated by Gerrit Code Review almost 9 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40608
Updated by Mathias Schreiber almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7129e2e55f60ff598616476b582a5ca1b36e3247.
Updated by Gerrit Code Review almost 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch master of project Teams/Security/TYPO3v4-Core has been pushed to the review server.
It is available at http://review.typo3.org/40633
Updated by Alexander Opitz almost 9 years ago
- Status changed from Under Review to Resolved
Reopened by accident.