use text/plain as default content-type in .json

vercel · Mar 27, 2017 · 96d3249 · 96d3249
1 parent 8012c9e
commit 96d3249
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/server.js b/lib/server.js
@@ -51,7 +51,7 @@ exports.json = (req, { limit = '1mb' } = {}) =>
   Promise.resolve().then(() => {
     const type = req.headers['content-type'];
     const length = req.headers['content-length'];
-    const encoding = typer.parse(type).parameters.charset;
+    const encoding = type === undefined ? 'text/plain' : typer.parse(type).parameters.charset;
 
     let str = rawBodyMap.get(req);
 

diff --git a/test/index.js b/test/index.js
@@ -586,3 +586,16 @@ test('res.end is working', async t => {
 
   t.deepEqual(res, 'woot');
 });
+
+test('json should throw 400 on empty body with no headers', async t => {
+  const fn = async req => json(req);
+
+  const url = await getUrl(fn);
+
+  try {
+    await request(url);
+  } catch (err) {
+    t.is(err.message, '400 - "Invalid body"');
+    t.is(err.statusCode, 400);
+  }
+});