Catch express bodyParser error

Question

Я хочу поймать ошибку из промежуточного программного обеспечения bodyParser() при отправке json-объекта, и это неверно, потому что я хочу отправить настраиваемый ответ вместо общей ошибки 400.

Это то, что у меня есть, и она работает:

app.use (express.bodyParser()); 
app.use (function (error, req, res, next){ 
    //Catch bodyParser error 
    if (error.message === "invalid json"){ 
     sendError (res, myCustomErrorMessage); 
    }else{ 
     next(); 
    } 
}); 

Но это мне кажется очень некрасиво подход, потому что я сравниваю сообщение об ошибке, которое может измениться в будущих версиях экспресс. Есть ли другой способ уловить ошибки bodyParser()?

EDIT:

Это ошибка, когда тело запроса имеет недопустимое JSON:

{ 
    stack: 'Error: invalid json\n at Object.exports.error (<path>/node_modules/express/node_modules/connect/lib/utils.js:55:13)\n at IncomingMessage.<anonymous> (<path>/node_modules/express/node_modules/connect/lib/middleware/json.js:74:71)\n at IncomingMessage.EventEmitter.emit (events.js:92:17)\n at _stream_readable.js:872:14\n at process._tickDomainCallback (node.js:459:13)', 
    arguments: undefined, 
    type: undefined, 
    message: 'invalid json', 
    status: 400 
} 

Довольно печататься стек:

Error: invalid json 
    at Object.exports.error (<path>/node_modules/express/node_modules/connect/lib/utils.js:55:13) 
    at IncomingMessage.<anonymous> (<path>/node_modules/express/node_modules/connect/lib/middleware/json.js:74:71) 
    at IncomingMessage.EventEmitter.emit (events.js:92:17) 
    at _stream_readable.js:872:14 
    at process._tickDomainCallback (node.js:459:13) 

Answer 1

ОК, нашел его:

bodyParser() - функция удобства для json(), urlencoded() и multipart(). Мне просто нужно вызвать json(), поймать ошибку и вызвать urlencoded() и multipart().

bodyParser source

app.use (express.json()); 
app.use (function (error, req, res, next){ 
    //Catch json error 
    sendError (res, myCustomErrorMessage); 
}); 

app.use (express.urlencoded()); 
app.use (express.multipart()); 

Answer 2

Я думаю, вам лучше всего, чтобы проверить SyntaxError:

app.use(function (error, req, res, next) { 
    if (error instanceof SyntaxError) { 
    sendError(res, myCustomErrorMessage); 
    } else { 
    next(); 
    } 
}); 

Answer 3

Я нашел проверки SyntaxError быть недостаточно, поэтому я делаю:

if (err instanceof SyntaxError && 
    err.status >= 400 && err.status < 500 && 
    err.message.indexOf('JSON')) { 
    // process filtered exception here 
} 

Answer 4

(bodyParser, req, res) => new Promise((resolve, reject) => { 
    try { 
     bodyParser(req, res, err => { 
      if (err instanceof Error) { 
       reject(err); 
      } else { 
       resolve(); 
      } 
     }); 
    } catch (e) { 
     reject(e); 
    } 
}) 

пуленепробиваемые. Перспективная известно. WTFPL-Лицензированный. А также полезно w/async/await.

Answer 5

Из ответа @alexander но с примером ussage

app.use((req, res, next) => { 
    bodyParser.json({ 
     verify: addRawBody, 
    })(req, res, (err) => { 
     if (err) { 
      console.log(err); 
      res.sendStatus(400); 
      return; 
     } 
     next(); 
    }); 
}); 

function addRawBody(req, res, buf, encoding) { 
    req.rawBody = buf.toString(); 
}