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 Error Handling #31

Closed
idky137 opened this issue May 24, 2024 · 2 comments
Closed

Add Error Handling #31

idky137 opened this issue May 24, 2024 · 2 comments
Labels
V1.0 Required for Zaino V1.0
Milestone

Comments

@idky137
Copy link
Contributor

idky137 commented May 24, 2024

Remove any unwraps that have the possibility of returning an error from production code.
Instead handle errors, either re-queuing requests or returning an error message to the client.
For serious system errors possibly implement graceful shutdown or soft restart.

@idky137 idky137 added this to the ZingoProxyD v1.0.0 milestone May 24, 2024
@idky137
Copy link
Contributor Author

idky137 commented Sep 10, 2024

Unwraps have been removed but there are still several "TODO's" that need to be replaced with correct error handling logic.

@idky137 idky137 added the V1.0 Required for Zaino V1.0 label Sep 10, 2024
@idky137
Copy link
Contributor Author

idky137 commented Sep 25, 2024

This work will be completed in #67

@idky137 idky137 closed this as completed Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V1.0 Required for Zaino V1.0
Projects
None yet
Development

No branches or pull requests

1 participant