Merge pull request routablehq#110 from adam-pro/download-pdf-auth-error-msg-bug

ej2 · web-flow · commit 786d5213b3a6 · 2018-10-18T11:51:19.000-07:00
Fixed bug in PDF download of lost error details on authentication failure
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -1,6 +1,9 @@
 Changelog
 ========
 
+* 0.7.5 (May 11th, 2018)
+    * Fixed bug with reporting authentication failure when attempting to download PDF (previously the error details were "lost")
+
 * 0.7.4 (March 26th, 2018)
     * Fixed bug in SendMixin send method.
     * Added support for send_to email to SendMixin.
diff --git a/quickbooks/client.py b/quickbooks/client.py
@@ -315,6 +315,11 @@ def download_pdf(self, qbbo, item_id):
         response = self.process_request("GET", url, headers=headers)
 
         if response.status_code != httplib.OK:
+
+            if response.status_code == httplib.UNAUTHORIZED:
+                # Note that auth errors have different result structure which can't be parsed by handle_exceptions()
+                raise AuthorizationException("Application authentication failed", detail=response.text)
+
             try:
                 result = response.json()
             except:
diff --git a/setup.py b/setup.py
@@ -10,7 +10,7 @@ def read(*parts):
         return fp.read()
 
 
-VERSION = (0, 7, 4)
+VERSION = (0, 7, 5)
 version = '.'.join(map(str, VERSION))
 
 setup(
