[java-identity-provider] branch main updated: Plug a logging hole when an exception is raised.

[Codeberg]

This is an automated email from the git hooks/post-receive script.

codeberg pushed a commit to branch main
in repository java-identity-provider.

View the commit online:
https://codeberg.org/Shibboleth/java-identity-provider/commit/f4167b8eaa138dffcaaaf3ced88c800777be4e55

The following commit(s) were added to refs/heads/main by this push:
     new f4167b8ea Plug a logging hole when an exception is raised.
f4167b8ea is described below

commit f4167b8eaa138dffcaaaf3ced88c800777be4e55
Author: Scott Cantor <scott at restingparrotsoftware.com>
AuthorDate: Tue Apr 7 15:48:05 2026 -0400

    Plug a logging hole when an exception is raised.
---
 .../idp/saml/saml1/profile/impl/AddAttributeStatementToAssertion.java    | 1 +
 .../idp/saml/saml2/profile/impl/AddAttributeStatementToAssertion.java    | 1 +
 2 files changed, 2 insertions(+)

diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml1/profile/impl/AddAttributeStatementToAssertion.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml1/profile/impl/AddAttributeStatementToAssertion.java
index b3b632394..b85423ca0 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml1/profile/impl/AddAttributeStatementToAssertion.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml1/profile/impl/AddAttributeStatementToAssertion.java
@@ -107,6 +107,7 @@ public class AddAttributeStatementToAssertion extends BaseAddAttributeStatementT
 
             log.debug("{} Adding constructed AttributeStatement to Assertion {} ", getLogPrefix(), assertion.getID());
         } catch (final AttributeEncodingException e) {
+            log.error("{} Failure encoding an IdPAttribute", getLogPrefix(), e);
             ActionSupport.buildEvent(profileRequestContext, IdPEventIds.UNABLE_ENCODE_ATTRIBUTE);
         }
     }
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAttributeStatementToAssertion.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAttributeStatementToAssertion.java
index d1ba0c804..6fa39ac51 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAttributeStatementToAssertion.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/AddAttributeStatementToAssertion.java
@@ -121,6 +121,7 @@ public class AddAttributeStatementToAssertion extends BaseAddAttributeStatementT
             
             log.debug("{} Adding constructed AttributeStatement to Assertion {} ", getLogPrefix(), id);
         } catch (final AttributeEncodingException e) {
+            log.error("{} Failure encoding an IdPAttribute", getLogPrefix(), e);
             ActionSupport.buildEvent(profileRequestContext, IdPEventIds.UNABLE_ENCODE_ATTRIBUTE);
         }
     }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.