Merge pull request #38 from bugout-dev/fix-network-errors-pr

Defined network error as 599

Author: kompotkot

bugout/__init__.py

@@ -7,7 +7,7 @@
 
 __email__ = "engineering@bugout.dev"
 __license__ = "MIT"
-__version__ = "0.1.19"
+__version__ = "0.1.20"
 
 __all__ = (
     "__author__",

bugout/calls.py

@@ -11,21 +11,23 @@ def make_request(method: Method, url: str, **kwargs) -> Any:
         response = requests.request(method.value, url=url, **kwargs)
         response.raise_for_status()
     except requests.exceptions.RequestException as err:
-        response = err.response
+        r = err.response
         if not err.response:
             # Connection errors, timepouts, etc...
-            raise BugoutResponseException("Network error", detail=str(err))
-        if response.headers.get("Content-Type") == "application/json":
-            exception_detail = response.json()["detail"]
+            raise BugoutResponseException(
+                "Network error", status_code=599, detail=str(err)
+            )
+        if r.headers.get("Content-Type") == "application/json":
+            exception_detail = r.json()["detail"]
         else:
-            exception_detail = response.text
+            exception_detail = r.text
         raise BugoutResponseException(
             "An exception occurred at Bugout API side",
-            status_code=response.status_code,
+            status_code=r.status_code,
             detail=exception_detail,
         )
     except Exception as e:
-        raise BugoutUnexpectedResponse({str(e)})
+        raise BugoutUnexpectedResponse(str(e))
     return response.json()
 
 

bugout/exceptions.py

@@ -21,7 +21,7 @@ class BugoutResponseException(Exception):
     def __init__(
         self,
         message,
-        status_code: Optional[int] = None,
+        status_code: int,
         detail: Optional[Any] = None,
     ) -> None:
         super().__init__(message)