2
0
Fork
You've already forked taskcollect
3

Implement HTTP handler functions and improve error logging #47

Merged
kvo merged 5 commits from rayokamoto/taskcollect:servemux into main 2022年11月21日 08:29:13 +01:00
Contributor
Copy link

Commit rayokamoto/taskcollect@48b2c59f39 implements HTTP handler functions, in an effort to better organise code.

Commit b6f31c1853 partially solves issue #34 by adding array bounds checking for getDirectDriveLink(). However, the whole codebase is yet to be checked whether array bounds checking is fully implemented.

Commit rayokamoto/taskcollect@f1a5915d30 brings error logging improvements, resolving the following issues:

  • fix(#28): Errors (and other information) are logged to the console and optionally, a log file. This has been implemented in previous commits.
  • fix(#18): Errors are wrapped at each stage and context has been added as to what the exact error is.
  • fix(#41): Error messages now yield more information (see #18)
Commit https://codeberg.org/rayokamoto/taskcollect/commit/48b2c59f39c09e1d0a74cbaac41f790a9686586d implements HTTP handler functions, in an effort to better organise code. Commit https://codeberg.org/kvo/taskcollect/commit/b6f31c1853b7bc7eea942f009d48382a54790756 partially solves issue #34 by adding array bounds checking for `getDirectDriveLink()`. However, the whole codebase is yet to be checked whether array bounds checking is fully implemented. Commit https://codeberg.org/rayokamoto/taskcollect/commit/f1a5915d30e7079498f5a7cf799fc4cc4e658d95 brings error logging improvements, resolving the following issues: - fix(#28): Errors (and other information) are logged to the console and optionally, a log file. This has been implemented in previous commits. - fix(#18): Errors are wrapped at each stage and context has been added as to what the exact error is. - fix(#41): Error messages now yield more information (see #18)
- Use dedicated handlers instead of using one root handler for more
 efficient handling of requests
- Clean up comments in main/errors and main/logger
- Use .AsError() only as needed when returning errors
This partially resolves issue #34 
fix(#28): Errors are logged to the console and optionally, a log file.
This has been implemented in previous commits.
fix(#18): Errors are wrapped at each stage and context has been added as
to what the exact error is.
fix(#41): Error messages now yield more information (see #18)
kvo referenced this pull request from a commit 2022年11月21日 08:29:13 +01:00
kvo referenced this pull request from a commit 2024年06月10日 13:33:34 +02:00
kvo referenced this pull request from a commit 2024年10月05日 16:12:04 +02:00
kvo referenced this pull request from a commit 2025年07月27日 12:40:52 +02:00
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
kvo/taskcollect!47
Reference in a new issue
kvo/taskcollect
No description provided.
Delete branch "rayokamoto/taskcollect:servemux"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?