Closed
Bug 1024007
Opened 12 years ago
Closed 12 years ago
Add a middleware to handle 503 errors
Categories
(Hello (Loop) :: Server, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: alexis+bugs, Assigned: rhubscher)
Details
Attachments
(1 file)
We're currently duplicating a lot of times code like that:
if (err) {
logError(err);
res.json(503, "Service Unavailable");
}
I believe this could add an utility function instead, and that we could call res.error(err) for instance, that would do the necessary things for us.
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → rhubscher
|
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #8441349 -
Flags: review?(alexis+bugs)
|
|
Assignee | |
Updated•12 years ago
|
Status: NEW → ASSIGNED
|
|
Reporter | |
Comment 2•12 years ago
|
||
Comment on attachment 8441349 [details] [review]
Link to github PR
r+wc
Attachment #8441349 -
Flags: review?(alexis+bugs) → review+
|
|
Reporter | |
Comment 3•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
|
||
Comment 4•12 years ago
|
||
Fix either went out with a previous deployment to Production or has been superceded by new code...
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•