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

Attribute all client errors as client errors and reset HTTP 5xx alarm back to 0 #374

Open
chris13524 opened this issue Feb 20, 2024 · 0 comments
Labels

Comments

@chris13524
Copy link
Member

Throughout the code there are many TODOs where client errors are currently thrown as server errors. These need to individually be investigated and each error attributed as a client or a server error. Eventually reducing the size of or even eliminating the NotifyServerError struct.

Once these are are all done, the HTTP 5xx alarm should be reset back to 0 so we know if there are any unexpected errors.

@chris13524 chris13524 self-assigned this Feb 20, 2024
@arein arein added the accepted label Feb 20, 2024
@chris13524 chris13524 removed their assignment Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants