Log server responses when something fails

softwarepub · Dec 20, 2023 · 60ecf7c · 60ecf7c
1 parent 0ac1c29
commit 60ecf7c
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/hermes/commands/deposit/invenio.py b/src/hermes/commands/deposit/invenio.py
@@ -274,6 +274,7 @@ def create_initial_version(self) -> None:
 
         response = self.client.new_deposit()
         if not response.ok:
+            _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
             raise RuntimeError(f"Could not create initial deposit {response.url!r}")
 
         deposit = response.json()
@@ -288,6 +289,7 @@ def create_new_version(self) -> None:
         # Get current deposit
         response = self.client.get_deposit(latest_record_id)
         if not response.ok:
+            _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
             raise RuntimeError(f"Failed to get current deposit {response.url!r}")
 
         self.links.update(response.json()["links"])
@@ -296,6 +298,7 @@ def create_new_version(self) -> None:
         deposit_url = self.links["newversion"]
         response = self.client.post(deposit_url)
         if not response.ok:
+            _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
             raise RuntimeError(f"Could not create new version deposit {deposit_url!r}")
 
         # Store link to latest draft to be used in :func:`update_metadata`.
@@ -315,6 +318,7 @@ def update_metadata(self) -> None:
         )
 
         if not response.ok:
+            _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
             raise RuntimeError(f"Could not update metadata of draft {draft_url!r}")
 
         deposit = response.json()
@@ -356,6 +360,7 @@ def upload_artifacts(self) -> None:
                     f"{bucket_url}/{path.name}", data=file_content,
                 )
                 if not response.ok:
+                    _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
                     raise RuntimeError(f"Could not upload file {path.name!r} into bucket {bucket_url!r}")
 
             # This can potentially be used to verify the checksum
@@ -367,7 +372,7 @@ def publish(self) -> None:
         publish_url = self.links["publish"]
         response = self.client.post(publish_url)
         if not response.ok:
-            _log.debug(response.text)
+            _log.error("Server answered with error code %d:\n%s", response.status_code, response.text)
             raise RuntimeError(f"Could not publish deposit via {publish_url!r}")
 
         record = response.json()