Merge branch 'feature/safety' into 'gray'
Feature/safety See merge request !1455
Showing
1 changed file
with
5 additions
and
3 deletions
@@ -106,9 +106,11 @@ exports.serverError = () => { | @@ -106,9 +106,11 @@ exports.serverError = () => { | ||
106 | 106 | ||
107 | forceNoCache(res); | 107 | forceNoCache(res); |
108 | 108 | ||
109 | - err.code = err.code || err.statusCode || 500; | 109 | + err = err || {}; |
110 | 110 | ||
111 | - if (!err.type === 'entity.parse.failed') { // json 解析失败不上报错误 | 111 | + err.code = parseInt(err.code || err.statusCode, 10) || 500; |
112 | + | ||
113 | + if (err.type !== 'entity.parse.failed') { // json 解析失败不上报错误 | ||
112 | logger.error(`error at path: ${req.url}`); | 114 | logger.error(`error at path: ${req.url}`); |
113 | logger.error(err); | 115 | logger.error(err); |
114 | } | 116 | } |
@@ -199,7 +201,7 @@ exports.serverError = () => { | @@ -199,7 +201,7 @@ exports.serverError = () => { | ||
199 | } | 201 | } |
200 | 202 | ||
201 | if (!res.headersSent) { | 203 | if (!res.headersSent) { |
202 | - return _err500(req, res, err.code, err); | 204 | + return _err500(req, res, 500, err); |
203 | } | 205 | } |
204 | 206 | ||
205 | return next(err); | 207 | return next(err); |
-
Please register or login to post a comment