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

Remove central exception handling advice #906

Open
orange-buffalo opened this issue Sep 10, 2023 · 0 comments
Open

Remove central exception handling advice #906

orange-buffalo opened this issue Sep 10, 2023 · 0 comments

Comments

@orange-buffalo
Copy link
Owner

orange-buffalo commented Sep 10, 2023

RestApiControllerExceptionsHandler is handling all possible exceptions, which couples it to implementation of different business processes.
We should move process-specific exceptions to the related controllers.
Makes sense to use ApiErrorHandler and HandleApiErrorsWith infrastructure.
We also should provide tests for common types of errors and generate API schema components for those DTOs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

1 participant