[java-identity-provider] branch main updated: Adjust logging to prevent complete dump of attribute objects.

[Scott Cantor]

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

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

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

The following commit(s) were added to refs/heads/main by this push:
       new  e15b50ed2 Adjust logging to prevent complete dump of attribute objects.
e15b50ed2 is described below

commit e15b50ed22ff7d77616f6f016816ff826f412bf9
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Jan 25 19:50:54 2021 -0500

    Adjust logging to prevent complete dump of attribute objects.
---
 .../idp/consent/flow/ar/impl/ReleaseAttributes.java       | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/ar/impl/ReleaseAttributes.java b/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/ar/impl/ReleaseAttributes.java
index ee7d5c995..282a4a19f 100644
--- a/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/ar/impl/ReleaseAttributes.java
+++ b/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/ar/impl/ReleaseAttributes.java
@@ -64,36 +64,37 @@ public class ReleaseAttributes extends AbstractAttributeReleaseAction {
         log.debug("{} Consents '{}'", getLogPrefix(), consents);
 
         final Map<String, IdPAttribute> attributes = getAttributeContext().getIdPAttributes();
-        log.debug("{} Attributes before release '{}'", getLogPrefix(), attributes);
+        log.debug("{} Attributes before release '{}'", getLogPrefix(), attributes.keySet());
 
         final Map<String, IdPAttribute> releasedAttributes = new HashMap<>(attributes.size());
 
         for (final IdPAttribute attribute : attributes.values()) {
             if (!getAttributeReleaseContext().getConsentableAttributes().containsKey(attribute.getId())) {
                 log.debug("{} Attribute '{}' will be released because it is excluded from consent", getLogPrefix(),
-                        attribute);
+                        attribute.getId());
                 releasedAttributes.put(attribute.getId(), attribute);
                 continue;
             }
             if (!consents.containsKey(attribute.getId())) {
                 log.debug("{} Attribute '{}' will not be released because consent for it does not exist",
-                        getLogPrefix(), attribute);
+                        getLogPrefix(), attribute.getId());
                 continue;
             }
             final Consent consent = consents.get(attribute.getId());
             if (consent.isApproved()) {
-                log.debug("{} Attribute '{}' will be released because consent is approved", getLogPrefix(), attribute);
+                log.debug("{} Attribute '{}' will be released because consent is approved", getLogPrefix(),
+                        attribute.getId());
                 releasedAttributes.put(attribute.getId(), attribute);
             } else {
                 log.debug("{} Attribute '{}' will not be released because consent is not approved", getLogPrefix(),
-                        attribute);
+                        attribute.getId());
             }
         }
 
         if (log.isDebugEnabled()) {
-            log.debug("{} Releasing attributes '{}'", getLogPrefix(), releasedAttributes);
+            log.debug("{} Releasing attributes: {}", getLogPrefix(), releasedAttributes.keySet());
             final MapDifference<String, IdPAttribute> diff = Maps.difference(attributes, releasedAttributes);
-            log.debug("{} Not releasing attributes '{}'", getLogPrefix(), diff.entriesOnlyOnLeft());
+            log.debug("{} Not releasing attributes: {}", getLogPrefix(), diff.entriesOnlyOnLeft().keySet());
         }
 
         getAttributeContext().setIdPAttributes(releasedAttributes.values());

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