Skip to content

Commit e5bd4ad

Browse files
Merge pull request #36 from open-runtimes/fix-error-handler
chore: update error handler
2 parents 30b5088 + ba7bdef commit e5bd4ad

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

app/http.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,12 @@ function logError(Throwable $error, string $action, ?Logger $logger, Route $rout
503503
->inject('response')
504504
->action(function (Http $utopia, throwable $error, ?Logger $logger, Request $request, Response $response) {
505505
$route = $utopia->match($request);
506-
logError($error, "httpError", $logger, $route);
506+
try {
507+
logError($error, "httpError", $logger, $route);
508+
} catch (Throwable) {
509+
Console::warning('Unable to send log message');
510+
}
511+
507512

508513
$version = (string) Http::getEnv('OPR_PROXY_VERSION') ?: 'UNKNOWN';
509514
$message = $error->getMessage();

composer.lock

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)