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

[Enhancement] Add global error handler #173

Open
Merseleo opened this issue Jan 22, 2025 · 0 comments
Open

[Enhancement] Add global error handler #173

Merseleo opened this issue Jan 22, 2025 · 0 comments
Labels
refactor Inner code structure is changed wontfix This will not be worked on

Comments

@Merseleo
Copy link
Contributor

Problem

Currently there is no consistent error handling in this frontend and in particular sometimes just any error messages are displayed, sometimes something is logged in the console, sometimes not, sometimes the error messages are shown directly to the user and sometimes nothing at all. This should be standardized and summarized in an GrpcErrorHandler.

Solution

The solution of this problem could be the implementation of a globally available error handler that can process a GRPC error object based on the status code, i.e.

  • log the error to the console, provided it does not reveal any sensitive data
  • map the error code to an error message if it is a standard error, such as no connection (this would display the same error message on all pages and in all cases for the same errors)
  • if an error code is not known and there is therefore a custom error, forwards the supplied message
  • in the default case (code unknown and no further information): “Unknown error” or similar.

This allows the error received after a request to be globally mapped to a message and displayed and the logging of errors is bundled in a central location.

@Merseleo Merseleo added the refactor Inner code structure is changed label Jan 22, 2025
@Merseleo Merseleo mentioned this issue Jan 22, 2025
5 tasks
@Merseleo Merseleo added the wontfix This will not be worked on label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Inner code structure is changed wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant