fix: more log for bodyParser (#3809)

eggjs · Jul 17, 2019 · 29a2f2f · 29a2f2f
1 parent 6dc8a2d
commit 29a2f2f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/config/config.default.js b/config/config.default.js
@@ -205,6 +205,10 @@ module.exports = appInfo => {
       depth: 5,
       parameterLimit: 1000,
     },
+    onerror(err) {
+      err.message += ', check bodyParser config';
+      throw err;
+    },
   };
 
   /**

diff --git a/test/app/middleware/body_parser.test.js b/test/app/middleware/body_parser.test.js
@@ -49,6 +49,15 @@ describe('test/app/middleware/body_parser.test.js', () => {
       .expect(200, done);
   });
 
+  it('should 413 when post json body over the limit', () => {
+    app.mockCsrf();
+    return app.httpRequest()
+      .post('/test/body_parser/user')
+      .send({ foo: 'a'.repeat(1024 * 200) })
+      .expect(/request entity too large, check bodyParser config/)
+      .expect(413);
+  });
+
   it('should disable body parser', async () => {
     app1 = utils.app('apps/body_parser_testapp_disable');
     await app1.ready();