thereadme/hapi-sequelize-crud
1
0
Fork 0
Code Issues 4 Pull Requests 1 Projects Releases Wiki Activity

Fix (error) always reply with an error #oops

Browse Source 
We had a case where reply would never be called. This could case a
server hang.
This commit is contained in:
Joey Baker 2016-09-04 17:25:57 -07:00
parent 79c6a81a3a
commit b032be20d1
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/error.js
View File

@ -14,14 +14,15 @@ export default (target, key, descriptor) => {
if (code && (code.startsWith('22') || code.startsWith('23'))) { if (code && (code.startsWith('22') || code.startsWith('23'))) {
const error = Boom.wrap(e, 406); const error = Boom.wrap(e, 406);
// detail tends to be more specific information. So, if we have it, use. // detail tends to be more specific information. So, if we have it, use.
if (detail) { if (detail) {
error.message += `: ${detail}`; error.message += `: ${detail}`;
error.reformat(); error.reformat();
}
reply(error);
} }
}
reply(error);
} else if (!e.isBoom) { } else if (!e.isBoom) {
const { message } = e; const { message } = e;
let err; let err;