don't fail when request content type is application/json but the body is empty

2 files changed, 12 insertions, 1 deletions

diff --git a/bottle.py b/bottle.py
index 8b2d88e..d6ea90b 100644
--- a/bottle.py
+++ b/bottle.py
@@ -1122,7 +1122,10 @@ class BaseRequest(object):
             exhaustion. """
         ctype = self.environ.get('CONTENT_TYPE', '').lower().split(';')[0]
         if ctype == 'application/json':
-            return json_loads(self._get_body_string())
+            b = self._get_body_string()
+            if not b:
+                return None
+            return json_loads(b)
         return None
 
     def _iter_body(self, read, bufsize):
diff --git a/test/test_environ.py b/test/test_environ.py
index c760f76..03ec86c 100755
--- a/test/test_environ.py
+++ b/test/test_environ.py
@@ -394,6 +394,14 @@ class TestRequest(unittest.TestCase):
         e['CONTENT_LENGTH'] = str(len(json_dumps(test)))
         self.assertEqual(BaseRequest(e).json, None)
 
+    def test_json_header_empty_body(self):
+        """Request Content-Type is application/json but body is empty"""
+        e = {'CONTENT_TYPE': 'application/json'}
+        wsgiref.util.setup_testing_defaults(e)
+        wsgiref.util.setup_testing_defaults(e)
+        e['CONTENT_LENGTH'] = "0"
+        self.assertEqual(BaseRequest(e).json, None)
+
     def test_isajax(self):
         e = {}
         wsgiref.util.setup_testing_defaults(e)