Merge pull request #250 from LD4P/alma-error-logging

Adds more logging to troubleshoot Alma errors
LD4P · Jan 9, 2025 · 28bcf17 · 28bcf17
2 parents 64a3fd6 + a6d1601
commit 28bcf17
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/ils_middleware/tasks/alma/post_bfinstance.py b/ils_middleware/tasks/alma/post_bfinstance.py
@@ -89,7 +89,9 @@ def NewInstancetoAlma(**kwargs):
             put_mms_id_str,
         )
     else:
-        raise Exception(f"Unexpected status code from Alma API: {status}")
+        raise Exception(
+            f"Unexpected status code from Alma API: {status}, Error detail {result}"
+        )
 
 
 def putInstanceToAlma(
@@ -117,5 +119,5 @@ def putInstanceToAlma(
             raise Exception(f"Internal server error from Alma API: {put_update_status}")
         case _:
             raise Exception(
-                f"Unexpected status code from Alma API: {put_update_status}"
+                f"Unexpected status code from Alma API: {put_update_status}, Error detail: {put_update.text}"
             )
diff --git a/ils_middleware/tasks/alma/post_bfwork.py b/ils_middleware/tasks/alma/post_bfwork.py
@@ -90,7 +90,9 @@ def NewWorktoAlma(**kwargs):
             put_mms_id_str,
         )
     else:
-        raise Exception(f"Unexpected status code from Alma API: {status}")
+        raise Exception(
+            f"Unexpected status code from Alma API: {status} Error detail: {result}"
+        )
 
 
 def putWorkToAlma(
@@ -118,5 +120,5 @@ def putWorkToAlma(
             raise Exception(f"Internal server error from Alma API: {put_update_status}")
         case _:
             raise Exception(
-                f"Unexpected status code from Alma API: {put_update_status}"
+                f"Unexpected status code from Alma API: {put_update_status}. Error detail: {put_update.text}"
             )