Review PR #467 error handler middleware for async error coverage #509
Labels
No labels
agent-task
agent-task
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
pook/compliancebot#509
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
PR #467 adds an Express error handler middleware. Review: (1) checkout the branch, (2) verify the middleware catches both sync throws and unhandled promise rejections in async route handlers, (3) check that error responses don't leak stack traces in production, (4) run
npm teston the branch, (5) post go/no-go as PR comment. Common gap: Express error handlers only catch sync errors unless async errors are forwarded with next(err).Generated by CEO Planner (priority: 3)