From ac5fd313401ab6cb60e4e34748b57bc952cccbe1 Mon Sep 17 00:00:00 2001 From: "egon w. stemle" Date: Mon, 7 Jul 2025 11:14:37 +0200 Subject: [PATCH] add verbose error output Be more helpful in case of errors, e.g. an error 500 (because of an extra '/' should not be misread as an "ConnectionError: Cannot authenticate to dspace!"). Output `r.url, r.status_code, r.text`: ``` ERROR: Requestes url:https://HOST/repository/server/api//authn/status; status code:500; text:{"timestamp" :"2025-07-07T08:53:24.119+00:00","status":500,"error":"Internal Server Error","message":"The request was rejected because the URL contained a potentiall y malicious String \"//\"","path":"/repository/server/api//authn/status"} ``` --- dspace_rest_client/client.py | 1 + 1 file changed, 1 insertion(+) diff --git a/dspace_rest_client/client.py b/dspace_rest_client/client.py index 339fd9a..2791588 100644 --- a/dspace_rest_client/client.py +++ b/dspace_rest_client/client.py @@ -157,6 +157,7 @@ def authenticate(self, retry=False): return r_json['authenticated'] # Default, return false + _logger.error(f'Requestes url:{r.url}; status code:{r.status_code}; text:{r.text}') return False def refresh_token(self):