Skip to content
Snippets Groups Projects
Commit 5166cca9 authored by Andreas Kienast's avatar Andreas Kienast
Browse files

[BUGFIX] Bail out if `AbstractExceptionHandler` cannot write logs

If an exception cannot get written into the database, e.g. due to
connectivity issues, exceptions are already caught. However, this
doesn't work on low-level errors that may arise if e.g. the DI container
is defunct, required by `ConnectionPool` internals.

The exception handling is changed to cover all `\Throwable`s.

Resolves: #103402
Releases: main
Change-Id: I52ce0e91e7f3297893928d6d1b79616b34e85798
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83471


Reviewed-by: default avatarChristoph Lehmann <christoph.lehmann@networkteam.com>
Reviewed-by: default avatarOliver Hader <oliver.hader@typo3.org>
Tested-by: default avatarcore-ci <typo3@b13.com>
Tested-by: default avatarBenjamin Franzke <ben@bnf.dev>
Tested-by: default avatarOliver Hader <oliver.hader@typo3.org>
Reviewed-by: default avatarAndreas Kienast <a.fernandez@scripting-base.de>
Reviewed-by: default avatarBenjamin Franzke <ben@bnf.dev>
Tested-by: default avatarAndreas Kienast <a.fernandez@scripting-base.de>
parent ffebe4a9
Branches
Tags
No related merge requests found
......@@ -129,7 +129,7 @@ abstract class AbstractExceptionHandler implements ExceptionHandlerInterface, Si
try {
// Write error message to sys_log table
$this->writeLog($logTitle . ': ' . $logMessage);
} catch (\Exception $exception) {
} catch (\Throwable $exception) {
}
}
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment