Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add default exception handler #54

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

PineappleIOnic
Copy link
Member

This PR adds a default exception handler to utopia that is used when the user has not defined their own

Copy link
Contributor

@Meldiron Meldiron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

src/App.php Outdated Show resolved Hide resolved
src/App.php Outdated Show resolved Hide resolved
src/App.php Outdated Show resolved Hide resolved
src/App.php Outdated
@@ -576,6 +576,21 @@ public function execute(Route $route, Request $request): self
}
}
} catch (\Throwable $e) {
if (empty(self::$errors['*'])) { // If no error handler is set then add a default one.
self::$errors['*'][] = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no longer $errors['*'] please check the new implementation of errors.

@christyjacob4
Copy link
Contributor

@PineappleIOnic please resolve the conflicts and check for the failing tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants