[java-identity-provider] branch master updated: Additional logging improvement.

[Scott Cantor]

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

scantor pushed a commit to branch master
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=3ceb6550ec357b2738ea5a417b3c2890e5d57ff7

The following commit(s) were added to refs/heads/master by this push:
       new  3ceb655   Additional logging improvement.
3ceb655 is described below

commit 3ceb6550ec357b2738ea5a417b3c2890e5d57ff7
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Apr 27 15:52:01 2020 -0400

    Additional logging improvement.
---
 .../net/shibboleth/idp/saml/metadata/OrganizationUIInfo.java   | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/metadata/OrganizationUIInfo.java b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/metadata/OrganizationUIInfo.java
index 95eff55..b9bc9e4 100644
--- a/idp-saml-api/src/main/java/net/shibboleth/idp/saml/metadata/OrganizationUIInfo.java
+++ b/idp-saml-api/src/main/java/net/shibboleth/idp/saml/metadata/OrganizationUIInfo.java
@@ -52,11 +52,12 @@ public class OrganizationUIInfo {
     private final Predicate<LocalizedName> nullLanguageString = new Predicate<>() {
         public boolean test(final LocalizedName u) {
             if (u.getXMLLang() == null) {
-                LOG.warn("String with value {} in <Organization> has no language associated, ignoring.", u.getValue());
+                LOG.warn("String with value {} in <{}> has no language associated, ignoring", u.getValue(),
+                        u.getElementQName().getLocalPart());
                 return false;
             }
             if (u.getValue()== null) {
-                LOG.warn("Ignoring empty string in <Organization/>");
+                LOG.warn("Ignoring empty <{}> element", u.getElementQName().getLocalPart());
                 return false;
             }
             return true;
@@ -67,11 +68,12 @@ public class OrganizationUIInfo {
     private final Predicate<LocalizedURI> nullLanguageURL = new Predicate<>() {
         public boolean test(final LocalizedURI u) {
             if (u.getXMLLang() == null) {
-                LOG.warn("URI with value {} in <Organization> has no language associated, ignoring.", u.getURI());
+                LOG.warn("URL with value {} in <{}> has no language associated, ignoring", u.getURI(),
+                        u.getElementQName().getLocalPart());
                 return false;
             }
             if (u.getURI() == null) {
-                LOG.warn("Ignoring empty URUI in <Organization/>", u.getURI());
+                LOG.warn("Ignoring empty <{}> element", u.getElementQName().getLocalPart());
                 return false;
             }
             return true;

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