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.

Sign up for GitHub

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

Jump to bottom

Implement error handling #1

Open
sm13 opened this issue Nov 6, 2024 · 0 comments
Open

Implement error handling #1

sm13 opened this issue Nov 6, 2024 · 0 comments
Assignees
@sm13
Labels
enhancement New feature or request

Comments

@sm13
Copy link
Owner

sm13 commented Nov 6, 2024

Error handling is currently only partially and poorly done. Do it overall and properly.

Create an error type to aggregate errors, fail gracefully rather than panic, etc.

If the logger is already in place, well, make good use of it.
@sm13 sm13 added the enhancement New feature or request label Nov 6, 2024
@sm13 sm13 self-assigned this Nov 6, 2024
@sm13 sm13 mentioned this issue Nov 7, 2024
Open
sm13 added a commit that referenced this issue Nov 9, 2024
@sm13
Simplify error handling in the articlehandler module. Adds to #1.
1a67f83 
This change does not fully implement #1, rather it is a first step in
the right direction. A new error_html() function is used for all the
map_err() from the previous implementation. To make that possible, a new
Error enum was born.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sm13 sm13

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sm13