PsrLogErrorHandler: use in logs message of original exception instead…

… of ApiException
contributte · Sep 24, 2019 · d6d0a6b · d6d0a6b
1 parent 43f6fde
commit d6d0a6b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/ErrorHandler/PsrLogErrorHandler.php b/src/ErrorHandler/PsrLogErrorHandler.php
@@ -32,7 +32,7 @@ public function handle(Throwable $error): ApiResponse
 		if ($error instanceof ApiException && ($previous = $error->getPrevious()) !== null) {
 			// Server error is expected to contain a real error while client error can contain just information, why client request failed
 			$level = $error instanceof ServerErrorException ? LogLevel::ERROR : LogLevel::DEBUG;
-			$this->logger->log($level, $error->getMessage(), ['exception' => $previous]);
+			$this->logger->log($level, $previous->getMessage(), ['exception' => $previous]);
 		}
 
 		return parent::handle($error);

diff --git a/tests/cases/ErrorHandler/PsrLogErrorHandler.phpt b/tests/cases/ErrorHandler/PsrLogErrorHandler.phpt
@@ -59,14 +59,14 @@ test(function (): void {
 		[
 			[
 				'level' => 'debug',
-				'message' => 'client',
+				'message' => 'test',
 				'context' => [
 					'exception' => $previousError,
 				],
 			],
 			[
 				'level' => 'error',
-				'message' => 'server',
+				'message' => 'test',
 				'context' => [
 					'exception' => $previousError,
 				],