Merge branch 'verbose-400' into 'master'

Make BadRequestErrors more verbose

See merge request !2

ai_web_common/rpc/core.py

@@ -27,11 +27,14 @@ HTTP_STATUS_CODE = attributes_helper.COMMON_ATTRIBUTES['HTTP_STATUS_CODE']
 
 class StatusError(Exception):
 
-    def __init__(self, url, resp):
+    def __init__(self, url, resp, extra_msg=None):
         self.url = url.url
         self.code = resp.status
-        super(StatusError, self).__init__('%s: HTTP error %d: %s' % (
-            self.url, self.code, resp.reason))
+        msg = '%s: HTTP error %d: %s' % (self.url, self.code, resp.reason)
+        if extra_msg:
+            msg += ' '
+            msg += extra_msg
+        super(StatusError, self).__init__(msg)
 
 
 class RetriableStatusError(StatusError):
@@ -41,16 +44,18 @@ class RetriableStatusError(StatusError):
 class BadRequestError(StatusError):
 
     def __init__(self, url, resp):
-        super(BadRequestError, self).__init__(url, resp)
-
         # Some backends return structured 400 errors with a
         # JSON-encoded body containing field-level validation
         # errors. Detect this and make it available in the 'errors'
         # attribute of the exception object.
         try:
             self.errors = _json_response_decoder(resp)
+            msg = json.dumps(self.errors)
         except ValueError:
             self.errors = {}
+            msg = None
+
+        super(BadRequestError, self).__init__(url, resp, extra_msg=msg)
 
 
 class AuthenticationError(StatusError):