Raise an exception when LCP fulfillment fails (#2033)

Raise an exception if we get an unexpected status code while doing a LCP fulfillment.
ThePalaceProject · Sep 6, 2024 · 9db1eeb · 9db1eeb
1 parent 5fb1411
commit 9db1eeb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/palace/manager/api/odl/api.py b/src/palace/manager/api/odl/api.py
@@ -567,7 +567,9 @@ def _lcp_fulfill(
         if content_link is None or content_type is None:
             raise CannotFulfill()
 
-        return FetchFulfillment(content_link, content_type)
+        return FetchFulfillment(
+            content_link, content_type, allowed_response_codes=["2xx"]
+        )
 
     def _bearer_token_fulfill(
         self, loan: Loan, delivery_mechanism: LicensePoolDeliveryMechanism

diff --git a/tests/manager/api/odl/test_api.py b/tests/manager/api/odl/test_api.py
@@ -894,6 +894,7 @@ def test_fulfill_success(
         assert isinstance(fulfillment, FetchFulfillment)
         assert correct_link == fulfillment.content_link
         assert correct_type == fulfillment.content_type
+        assert ["2xx"] == fulfillment.allowed_response_codes
 
     def test_fulfill_open_access(
         self,