Hardcode 502 in APIBadGateway instead of accepting a status_code

The class definitionally represents a 502, so there is no reason for
construction sites to pass (or be able to override) the status.

Author: mtorp

socketdev/core/api.py

@@ -112,7 +112,7 @@ def format_headers(headers_dict):
                 raise APIInsufficientQuota(status_code=429)
             if response.status_code == 502:
                 log.error(f"Upstream server error{path_str}{headers_str}")
-                raise APIBadGateway(status_code=502)
+                raise APIBadGateway()
             if response.status_code >= 400:
                 try:
                     error_json = response.json()

socketdev/exceptions.py

@@ -76,5 +76,5 @@ def is_transient_error(self) -> bool:
 class APIBadGateway(APIFailure):
     """Raised when the upstream server returns a 502 Bad Gateway error"""
 
-    def __init__(self, *args, status_code: Optional[int] = 502):
-        super().__init__(*args, status_code=status_code)
+    def __init__(self, *args):
+        super().__init__(*args, status_code=502)