lib: catch busboy parsing errors

lib/make-middleware.js

@@ -153,6 +153,7 @@ function makeMiddleware (setup) {
 
     })
 
+    busboy.on('error', function (err) { abortWithError(err) })
     busboy.on('partsLimit', function () { abortWithCode('LIMIT_PART_COUNT') })
     busboy.on('filesLimit', function () { abortWithCode('LIMIT_FILE_COUNT') })
     busboy.on('fieldsLimit', function () { abortWithCode('LIMIT_FIELD_COUNT') })

package.json

@@ -20,7 +20,7 @@
   ],
   "dependencies": {
     "append-field": "^0.1.0",
-    "busboy": "^0.2.9",
+    "busboy": "^0.2.11",
     "concat-stream": "^1.5.0",
     "mkdirp": "^0.5.1",
     "object-assign": "^3.0.0",

test/error-handling.js

@@ -177,4 +177,30 @@ describe('Error Handling', function () {
       done()
     })
   })
+
+  it('should report errors from busboy parsing', function (done) {
+    var req = new stream.PassThrough()
+    var storage = multer.memoryStorage()
+    var upload = multer({ storage: storage }).single('tiny0')
+    var boundary = 'AaB03x'
+    var body = [
+      '--' + boundary,
+      'Content-Disposition: form-data; name="tiny0"; filename="test.txt"',
+      'Content-Type: text/plain',
+      '',
+      'test without end boundary'
+    ].join('\r\n')
+
+    req.headers = {
+      'content-type': 'multipart/form-data; boundary=' + boundary,
+      'content-length': body.length
+    }
+
+    req.end(body)
+
+    upload(req, null, function (err) {
+      assert.equal(err.message, 'Unexpected end of multipart data')
+      done()
+    })
+  })
 })