Merge pull request #98 from maxmind/greg/release

2.7.0

Author: horgh

HISTORY.rst

@@ -3,7 +3,20 @@
 History
 -------
 
-2.7.0
+2.7.2 (2022-03-29)
+++++++++++++++++++
+
+* Updated code to correctly handle ``None`` ``Content-Type`` from minFraud
+  web service. This should never happen, but if it does, the correct
+  exception will now be thrown.
+
+2.7.1 (2022-03-29)
+++++++++++++++++++
+
+* Fixed ``KeyError`` when using the ``report()`` method. Reported by siang.
+  GitHub #99.
+
+2.7.0 (2022-03-28)
 ++++++++++++++++++
 
 * Added the input ``/credit_card/country``. This is the country where the

minfraud/version.py

@@ -1,2 +1,2 @@
 """Internal module for version (to prevent cyclic imports)"""
-__version__ = "2.6.0"
+__version__ = "2.7.2"

minfraud/webservice.py

@@ -83,7 +83,7 @@ def _handle_success(
         return model_class(decoded_body)  # type: ignore
 
     def _exception_for_error(
-        self, status: int, content_type: str, raw_body: str, uri: str
+        self, status: int, content_type: Optional[str], raw_body: str, uri: str
     ) -> Union[
         AuthenticationError,
         InsufficientFundsError,
@@ -100,7 +100,7 @@ def _exception_for_error(
         return self._exception_for_unexpected_status(status, raw_body, uri)
 
     def _exception_for_4xx_status(
-        self, status: int, content_type: str, raw_body: str, uri: str
+        self, status: int, content_type: Optional[str], raw_body: str, uri: str
     ) -> Union[
         AuthenticationError,
         InsufficientFundsError,
@@ -113,7 +113,7 @@ def _exception_for_4xx_status(
             return HTTPError(
                 f"Received a {status} error with no body", status, uri, raw_body
             )
-        if content_type.find("json") == -1:
+        if content_type is None or content_type.find("json") == -1:
             return HTTPError(
                 f"Received a {status} with the following body: {raw_body}",
                 status,
@@ -618,7 +618,7 @@ def report(self, report: Dict[str, Optional[str]], validate: bool = True) -> Non
 
         response = self._do_request(uri, prepared_request)
         status = response.status_code
-        content_type = response.headers["Content-Type"]
+        content_type = response.headers.get("Content-Type")
         raw_body = response.text
         if status != 204:
             raise self._exception_for_error(status, content_type, raw_body, uri)
@@ -636,7 +636,7 @@ def _response_for(
 
         response = self._do_request(uri, prepared_request)
         status = response.status_code
-        content_type = response.headers["Content-Type"]
+        content_type = response.headers.get("Content-Type")
         raw_body = response.text
         if status != 200:
             raise self._exception_for_error(status, content_type, raw_body, uri)