Merge pull request meraki#75 from coreGreenberet/aio_exception_problem_73

Aio exception problem 73

Author: Shiyue Cheng

.gitignore

@@ -1,4 +1,5 @@
 .DS_Store
 # No PyCharm config files
 .idea/
-venv/
+venv/
+__pycache__

meraki/aio/rest_session.py

@@ -89,6 +89,7 @@ async def request(self, metadata, method, url, **kwargs):
             return None
         else:
             response = None
+            message = None
             for _ in range(retries):
                 # Make the HTTP request to the API endpoint
                 try:
@@ -154,7 +155,7 @@ async def request(self, metadata, method, url, **kwargs):
                     self._logger.error(
                         f"{tag}, {operation} - {status} {reason}, {message}"
                     )
-            raise AsyncAPIError(metadata, response, await response.text())
+                    raise AsyncAPIError(metadata, response, message)
 
     async def get(self, metadata, url, params=None):
         metadata["method"] = "GET"

meraki/exceptions.py

@@ -31,12 +31,10 @@ def __init__(self, metadata, response, message):
         self.response = response
         self.tag = metadata['tags'][0]
         self.operation = metadata['operation']
-        self.status = self.response.status if self.response is not None and self.response.status else None
-        self.reason = self.response.reason if self.response is not None and self.response.reason else None
-        try:
-            self.message = self.response.json() if self.response is not None and self.response.json() else None
-        except ValueError:
-            self.message = self.response.text[:100]
+        self.status = response.status if response is not None and response.status else None
+        self.reason = response.reason if response is not None and response.reason else None
+        self.message = message
+
         super().__init__(
             f'{self.tag}, {self.operation} - {self.status} {self.reason}, {self.message}'
         )