[java-opensaml] branch main updated: Make error messages a bit more consistent.

Scott Cantor cantor.2 at osu.edu
Mon Oct 7 20:32:18 UTC 2024


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

scantor pushed a commit to branch main
in repository java-opensaml.

View the commit online:
http://git.shibboleth.net/view/?p=java-opensaml.git;a=commit;h=fd1b87fdd40ecd33717328cde1aaa7c2e27e4a4b

The following commit(s) were added to refs/heads/main by this push:
     new fd1b87fdd Make error messages a bit more consistent.
fd1b87fdd is described below

commit fd1b87fdd40ecd33717328cde1aaa7c2e27e4a4b
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Oct 7 16:32:13 2024 -0400

    Make error messages a bit more consistent.
---
 .../assertion/impl/AbstractSubjectConfirmationValidator.java      | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/assertion/impl/AbstractSubjectConfirmationValidator.java b/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/assertion/impl/AbstractSubjectConfirmationValidator.java
index d589f7fde..f84edb8c7 100644
--- a/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/assertion/impl/AbstractSubjectConfirmationValidator.java
+++ b/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/assertion/impl/AbstractSubjectConfirmationValidator.java
@@ -276,7 +276,7 @@ public abstract class AbstractSubjectConfirmationValidator implements SubjectCon
         }
         
         context.getValidationFailureMessages().add(String.format(
-                "Subject confirmation InResponseTo for assertion '%s' did not match the valid value: %s",
+                "SubjectConfirmationData/@InResponseTo for assertion '%s' did not match the valid value: %s",
                 assertion.getID(), validInResponseTo));
         return ValidationResult.INVALID;
     }
@@ -313,7 +313,7 @@ public abstract class AbstractSubjectConfirmationValidator implements SubjectCon
                 notBefore, skewedNow);
         if (notBefore != null && notBefore.isAfter(skewedNow)) {
             context.getValidationFailureMessages().add(String.format(
-                    "Subject confirmation, in assertion '%s', with NotBefore condition of '%s' is not yet valid",
+                    "SubjectConfirmationData in assertion '%s', with NotBefore condition of '%s' is not yet valid",
                     assertion.getID(), notBefore));
             return ValidationResult.INVALID;
         }
@@ -353,7 +353,7 @@ public abstract class AbstractSubjectConfirmationValidator implements SubjectCon
                 notOnOrAfter, skewedNow);
         if (notOnOrAfter != null && notOnOrAfter.isBefore(skewedNow)) {
             context.getValidationFailureMessages().add(String.format(
-                    "Subject confirmation, in assertion '%s', with NotOnOrAfter condition of '%s' is no longer valid",
+                    "SubjectConfirmationData, in assertion '%s', with NotOnOrAfter condition of '%s' is no longer valid",
                     assertion.getID(), notOnOrAfter));
             return ValidationResult.INVALID;
         }
@@ -412,7 +412,7 @@ public abstract class AbstractSubjectConfirmationValidator implements SubjectCon
         }
 
         context.getValidationFailureMessages().add(String.format(
-                "Subject confirmation recipient for assertion '%s' did not match any valid recipients: %s",
+                "SubjectConfirmationData/@Recipient for assertion '%s' did not match any valid recipients: %s",
                 assertion.getID(), validRecipients));
         return ValidationResult.INVALID;
     }

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


More information about the commits mailing list