small adjustment to error logging. The 'DocStatus' error occurs while…

… working with already committed Avalara document codes. These should be returned as errors to the codebase, however should not be logged as exceptions that need to be watched. ver bump

pyavatax/__init__.py

@@ -1,4 +1,4 @@
-VERSION = (1, 3, 2)
+VERSION = (1, 3, 3)
 
 
 from .api import API

pyavatax/api.py

@@ -1,21 +1,41 @@
 import decorator
+import json
 from pyavatax.base import Document, Address, BaseResponse, BaseAPI, AvalaraException, AvalaraTypeException, AvalaraValidationException, AvalaraServerException, ErrorResponse
 
 
 @decorator.decorator
 def except_500_and_return(fn, *args, **kwargs):
-    try:
-        return fn(*args, **kwargs)
-    except AvalaraServerException as e:
-        self = args[0]  # the first arg is self
-        self.logger.error(e.full_request_as_string)
+
+    def error_as_resp(args, e):
+        """
+        Always return the error wrapped in a response object
+        """
         resp = ErrorResponse(e.response)
         for arg in args:
             if isinstance(arg, Document):
                 self.recorder.failure(arg, resp)
                 break
         return resp
 
+    try:
+        return fn(*args, **kwargs)
+    except AvalaraServerException as e:
+        self = args[0]  # the first arg is self
+        logged = False
+        try:
+            # but don't log the doc status error as an exception
+            error_as_json = json.loads(e.full_request_as_string)
+            if 'DocStatus' in error_as_json:
+                if error_as_json == 'DocStatus is invalid for this operation.':
+                    self.logger.warning(e.full_request_as_string)  # this case is not an error, just log a warning
+                    logged = True
+
+            if not logged:
+                self.logger.exception(e.full_request_as_string)
+            return error_as_resp(args, e)
+        except ValueError:  # json failed to parse
+            self.logger.exception(e.full_request_as_string)
+            return error_as_resp(args, e)
 
 class API(BaseAPI):