From b80a5ae8380ac879d262d480a518842555c4244c Mon Sep 17 00:00:00 2001 From: Jonathan Green Date: Tue, 17 Mar 2026 14:50:57 -0300 Subject: [PATCH] Log PatronAuthResult details on server-side validation failure When server_side_validation() fails, the PatronAuthResult contains the failure reason but was previously discarded. Now capture and log the result details at INFO level to aid debugging authentication issues. --- src/palace/manager/api/authentication/basic.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/palace/manager/api/authentication/basic.py b/src/palace/manager/api/authentication/basic.py index 2ea21b4c91..cb03cabd48 100644 --- a/src/palace/manager/api/authentication/basic.py +++ b/src/palace/manager/api/authentication/basic.py @@ -506,7 +506,13 @@ def authenticate( """ username = self.scrub_credential(credentials.get("username")) or "" password = self.scrub_credential(credentials.get("password")) - if not self.server_side_validation(username, password): + result = self.server_side_validation(username, password) + if not result: + self.log.info( + "Server-side validation failed for identifier %r: %s", + username, + result.details, + ) return None # Check these credentials with the source of truth.