Fix recording of reponse headers in exceptions. (openai#44)

max-schaefer · web-flow · commit 7ea5dde96e58 · 2022-01-19T14:24:27.000-08:00
These were passed as positional arguments, but in the wrong position, which meant they were passed into the `json_body` parameter of the `OpenAIError` constructor.
diff --git a/openai/api_requestor.py b/openai/api_requestor.py
@@ -300,15 +300,15 @@ def _interpret_response_line(
     ) -> OpenAIResponse:
         if rcode == 503:
             raise error.ServiceUnavailableError(
-                "The server is overloaded or not ready yet.", rbody, rcode, rheaders
+                "The server is overloaded or not ready yet.", rbody, rcode, headers=rheaders
             )
         try:
             if hasattr(rbody, "decode"):
                 rbody = rbody.decode("utf-8")
             data = json.loads(rbody)
         except (JSONDecodeError, UnicodeDecodeError):
             raise error.APIError(
-                f"HTTP code {rcode} from API ({rbody})", rbody, rcode, rheaders
+                f"HTTP code {rcode} from API ({rbody})", rbody, rcode, headers=rheaders
             )
         resp = OpenAIResponse(data, rheaders)
         # In the future, we might add a "status" parameter to errors
