Log JSONDecodeErrors

docs/history.rst

@@ -3,6 +3,10 @@
 Version History
 ===============
+0.7.1 (2017-06-02)
+------------------
+ - Getting error response back that we can't decode, log it
+
 0.7.0 (2017-06-02)
 ------------------
  - Don't overwrite the ``Host`` HTTP header if it's set

setup.py

@@ -39,7 +39,7 @@ def read_requirements(name):
 setuptools.setup(
     name='tornado-aws',
-    version='0.7.0',
+    version='0.7.1',
     description=DESC,
     long_description=open('README.rst').read(),
     author='Gavin M. Roy',

tornado_aws/__init__.py

@@ -6,6 +6,6 @@ from tornado_aws.client import AWSClient
 from tornado_aws.client import AsyncAWSClient
 from tornado_aws.client import exceptions
-__version__ = '0.7.0'
+__version__ = '0.7.1'
 __all__ = ['AWSClient', 'AsyncAWSClient', 'exceptions']

tornado_aws/client.py

@@ -184,7 +184,12 @@ class AWSClient(object):
         if not isinstance(error, httpclient.HTTPError):
             return
         if error.code == 400:
-            payload = json.loads(error.response.body.decode('utf-8'))
+            try:
+                payload = json.loads(error.response.body.decode('utf-8'))
+            except json.JSONDecodeError as error:
+                LOGGER.error('Error decoding response: %r [%r]',
+                             error.response, error.response.body)
+                return
             if isinstance(payload, dict) and '__type' in payload:
                 return payload