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: bytes@3.0.0
   * deps: debug@2.6.8

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')