[java-shib-attribute] 02/02: JSATTR-6: SAML AttributeQuery DataConnector

Codeberg noreply at shibboleth.net
Fri Jan 16 01:01:22 UTC 2026


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

codeberg pushed a commit to branch main
in repository java-shib-attribute.

View the commit online:
https://codeberg.org/Shibboleth/java-shib-attribute/commit/5af3ebeb08f03a32069cc31fd8f883ca69a3e19e

commit 5af3ebeb08f03a32069cc31fd8f883ca69a3e19e
Author: Brent Putman <putmanb at georgetown.edu>
AuthorDate: Thu Jan 15 20:00:28 2026 -0500

    JSATTR-6: SAML AttributeQuery DataConnector
    
    Add some missing prefixes in log output.
---
 .../idp/attribute/resolver/dc/saml/impl/SAMLDataConnector.java    | 8 ++++----
 .../resolver/dc/saml/impl/SimpleAggregationSAMLDataConnector.java | 7 ++++---
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SAMLDataConnector.java b/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SAMLDataConnector.java
index dded0425e..f38566c7f 100644
--- a/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SAMLDataConnector.java
+++ b/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SAMLDataConnector.java
@@ -193,8 +193,8 @@ public class SAMLDataConnector extends AbstractSearchDataConnector<ExecutableQue
             throw new ComponentInitializationException("SubjectMatchProcessor was null");
         }
         
-        log.trace("Attribute dependencies: {}", getAttributeDependencies());
-        log.trace("Data Connector dependencies: {}", getDataConnectorDependencies());
+        log.trace("{} Attribute dependencies: {}", getLogPrefix(), getAttributeDependencies());
+        log.trace("{} Data Connector dependencies: {}", getLogPrefix(), getDataConnectorDependencies());
     }
 
     /** {@inheritDoc} */
@@ -269,10 +269,10 @@ public class SAMLDataConnector extends AbstractSearchDataConnector<ExecutableQue
         }
         
         if (isSubjectMatch()) {
-            log.debug("Subject match is enabled, processing");
+            log.debug("{} Subject match is enabled, processing", getLogPrefix());
             getSubjectMatchProcessor().process(responseData);
         } else {
-            log.debug("Subject match is disabled, skipping");
+            log.debug("{} Subject match is disabled, skipping", getLogPrefix());
             return;
         }
     }
diff --git a/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SimpleAggregationSAMLDataConnector.java b/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SimpleAggregationSAMLDataConnector.java
index 8ae00ed62..8e225cb58 100644
--- a/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SimpleAggregationSAMLDataConnector.java
+++ b/shib-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/saml/impl/SimpleAggregationSAMLDataConnector.java
@@ -119,8 +119,8 @@ public class SimpleAggregationSAMLDataConnector extends AbstractDataConnector {
             throw new ComponentInitializationException("EntityID sources list was empty");
         }
         
-        log.trace("Attribute dependencies: {}", getAttributeDependencies());
-        log.trace("Data Connector dependencies: {}", getDataConnectorDependencies());
+        log.trace("{} Attribute dependencies: {}", getLogPrefix(), getAttributeDependencies());
+        log.trace("{} Data Connector dependencies: {}", getLogPrefix(), getDataConnectorDependencies());
     }
 
     /** {@inheritDoc} */
@@ -140,7 +140,8 @@ public class SimpleAggregationSAMLDataConnector extends AbstractDataConnector {
         for (final String authorityEntityID : authtorityEntityIDs) {
             assert authorityEntityID != null;
             try {
-                log.debug("Executing SAML attribute query against authority entityID: {}", authorityEntityID);
+                log.debug("{} Executing SAML attribute query against authority entityID: {}",
+                        getLogPrefix(), authorityEntityID);
                 resolutionContext.addSubcontext(new AttributeAuthorityEntityIDContext(authorityEntityID), true);
                 final Map<String,IdPAttribute> authorityResults = getQueryConnector().resolve(resolutionContext);
                 if (authorityResults != null) {

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


More information about the commits mailing list