[java-opensaml] branch main updated: Add some quoting of interpolated values in log and exception messages.

[Brent Putman]

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

putmanb 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=9433c27728287cbce68fd732fa771dbe88cef0c4

The following commit(s) were added to refs/heads/main by this push:
     new 9433c2772 Add some quoting of interpolated values in log and exception messages.
9433c2772 is described below

commit 9433c27728287cbce68fd732fa771dbe88cef0c4
Author: Brent Putman <putmanb at georgetown.edu>
AuthorDate: Thu Aug 31 19:41:50 2023 -0400

    Add some quoting of interpolated values in log and exception messages.
---
 .../org/opensaml/core/xml/io/AbstractXMLObjectUnmarshaller.java   | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/opensaml-core-api/src/main/java/org/opensaml/core/xml/io/AbstractXMLObjectUnmarshaller.java b/opensaml-core-api/src/main/java/org/opensaml/core/xml/io/AbstractXMLObjectUnmarshaller.java
index 85d5dd019..f17a70689 100644
--- a/opensaml-core-api/src/main/java/org/opensaml/core/xml/io/AbstractXMLObjectUnmarshaller.java
+++ b/opensaml-core-api/src/main/java/org/opensaml/core/xml/io/AbstractXMLObjectUnmarshaller.java
@@ -402,11 +402,11 @@ public abstract class AbstractXMLObjectUnmarshaller implements Unmarshaller {
             throws UnmarshallingException {
 
         if (strictMode) {
-            throw new UnmarshallingException(String.format("Saw invalid attribute %s on element %s",
+            throw new UnmarshallingException(String.format("Saw invalid attribute '%s' on element %s",
                     QNameSupport.getNodeQName(attribute), xmlObject.getElementQName()));
         }
 
-        log.debug("Ignoring unknown attribute {} on element {}", QNameSupport.getNodeQName(attribute),
+        log.debug("Ignoring unknown attribute '{}' on element {}", QNameSupport.getNodeQName(attribute),
                 xmlObject.getElementQName());
     }
 
@@ -424,11 +424,11 @@ public abstract class AbstractXMLObjectUnmarshaller implements Unmarshaller {
             throws UnmarshallingException {
 
         if (strictMode) {
-            throw new UnmarshallingException(String.format("Saw invalid element content %s of element %s",
+            throw new UnmarshallingException(String.format("Saw invalid element content '%s' of element %s",
                     elementContent, xmlObject.getElementQName()));
         }
 
-        log.debug("Ignoring unknown element content {} of element {}", elementContent, xmlObject.getElementQName());
+        log.debug("Ignoring unknown element content '{}' of element {}", elementContent, xmlObject.getElementQName());
     }
     
     /**

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