Include the "body" property on verify errors

HISTORY.md

@@ -2,6 +2,7 @@ unreleased
 ==========
 
  * Fix JSON strict violation error to match native parse error
+ * Include the `body` property on verify errors
  * Use `http-errors` to set status code on errors
  * deps: [email protected]
  * deps: [email protected]

lib/read.js

@@ -96,26 +96,25 @@ function read (req, res, next, parse, debug, options) {
  debug('verify body')
  verify(req, res, body, encoding)
  } catch (err) {
- next(createError(403, err))
+ next(createError(403, err, {
+ body: body
+ }))
  return
  }
  }
 
  // parse
- var str
+ var str = body
  try {
  debug('parse body')
  str = typeof body !== 'string' && encoding !== null
  ? iconv.decode(body, encoding)
  : body
  req.body = parse(str)
  } catch (err) {
- // istanbul ignore next
- err.body = str === undefined
- ? body
- : str
-
- next(createError(400, err))
+ next(createError(400, err, {
+ body: str
+ }))
  return
  }
 

test/json.js

@@ -344,6 +344,19 @@ describe('bodyParser.json()', function () {
  .expect(400, 'no arrays', done)
  })
 
+ it('should include original body on error object', function (done) {
+ var server = createServer({verify: function (req, res, buf) {
+ if (buf[0] === 0x5b) throw new Error('no arrays')
+ }})
+
+ request(server)
+ .post('/')
+ .set('Content-Type', 'application/json')
+ .set('X-Error-Property', 'body')
+ .send('["tobi"]')
+ .expect(403, '["tobi"]', done)
+ })
+
  it('should allow pass-through', function (done) {
  var server = createServer({verify: function (req, res, buf) {
  if (buf[0] === 0x5b) throw new Error('no arrays')