[java-oidfed-common] 02/05: Removed setting of relying party ID and enabling the verified-flag.

Codeberg noreply at shibboleth.net
Wed Sep 23 07:34:31 UTC 2026


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

codeberg pushed a commit to branch main
in repository java-oidfed-common.

View the commit online:
https://codeberg.org/Shibboleth/java-oidfed-common/commit/8523372f1ad0a6dbe37a74bf44188eb09a4b6438

commit 8523372f1ad0a6dbe37a74bf44188eb09a4b6438
Author: Henri Mikkonen <henri.mikkonen at iki.fi>
AuthorDate: Wed Sep 23 09:58:08 2026 +0300

    Removed setting of relying party ID and enabling the verified-flag.
    
    - Already done earlier by the flow
---
 .../ValidateResolveEntityProfileConfiguration.java | 29 ----------------------
 1 file changed, 29 deletions(-)

diff --git a/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/profile/impl/ValidateResolveEntityProfileConfiguration.java b/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/profile/impl/ValidateResolveEntityProfileConfiguration.java
index a2cf306..335e9a7 100644
--- a/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/profile/impl/ValidateResolveEntityProfileConfiguration.java
+++ b/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/profile/impl/ValidateResolveEntityProfileConfiguration.java
@@ -29,12 +29,10 @@ import org.opensaml.profile.context.navigate.InboundMessageContextLookup;
 import org.slf4j.Logger;
 
 import net.shibboleth.idp.profile.AbstractProfileAction;
-import net.shibboleth.idp.profile.IdPEventIds;
 import net.shibboleth.oidfed.metadata.EntityStatement;
 import net.shibboleth.oidfed.profile.config.navigate.MandatoryTrustMarksLookupFunction;
 import net.shibboleth.oidfed.profile.context.RelyingPartyTrustChainContext;
 import net.shibboleth.oidfed.profile.context.VerifiedTrustChain;
-import net.shibboleth.profile.context.RelyingPartyContext;
 import net.shibboleth.shared.annotation.constraint.NonnullAfterInit;
 import net.shibboleth.shared.annotation.constraint.NonnullBeforeExec;
 import net.shibboleth.shared.annotation.constraint.NotEmpty;
@@ -61,9 +59,6 @@ public class ValidateResolveEntityProfileConfiguration extends AbstractProfileAc
     /** Class logger. */
     @Nonnull private Logger log = LoggerFactory.getLogger(ValidateResolveEntityProfileConfiguration.class);
 
-    /** Strategy that will return or create a {@link RelyingPartyContext}. */
-    @Nonnull private Function<ProfileRequestContext, RelyingPartyContext> relyingPartyContextCreationStrategy;
-
     /** Strategy used to lookup the trust chain context. */
     @Nonnull private Function<ProfileRequestContext, RelyingPartyTrustChainContext> trustChainContextLookupStrategy;
 
@@ -86,7 +81,6 @@ public class ValidateResolveEntityProfileConfiguration extends AbstractProfileAc
      * Constructor.
      */
     public ValidateResolveEntityProfileConfiguration() {
-        relyingPartyContextCreationStrategy = new ChildContextLookup<>(RelyingPartyContext.class, true);
         final Function<ProfileRequestContext, RelyingPartyTrustChainContext> tcls =
                 new ChildContextLookup<>(RelyingPartyTrustChainContext.class).compose(
                         new InboundMessageContextLookup());
@@ -95,19 +89,6 @@ public class ValidateResolveEntityProfileConfiguration extends AbstractProfileAc
         mandatoryTrustMarksLookupStrategy = new MandatoryTrustMarksLookupFunction();
     }
 
-    /**
-     * Set the strategy used to return or create the {@link RelyingPartyContext} .
-     * 
-     * @param strategy creation strategy
-     */
-    public void setRelyingPartyContextCreationStrategy(
-            @Nonnull final Function<ProfileRequestContext, RelyingPartyContext> strategy) {
-        ifInitializedThrowUnmodifiabledComponentException();
-
-        relyingPartyContextCreationStrategy =
-                Constraint.isNotNull(strategy, "RelyingPartyContext creation strategy cannot be null");
-    }
-
     /**
      * Set the strategy used to lookup the trust chain context.
      * 
@@ -201,16 +182,6 @@ public class ValidateResolveEntityProfileConfiguration extends AbstractProfileAc
         final List<EntityStatement<?>> trustChain = selectedTrustChain.getTrustChain();
         assert trustChain != null;
         trustChainContext.setSelectedMetadataExpiration(resolveTrustChainExpiration(trustChain));
-
-        final RelyingPartyContext rpContext = relyingPartyContextCreationStrategy.apply(profileRequestContext);
-        if (rpContext == null) {
-            log.error("{} Unable to locate or create RelyingPartyContext", getLogPrefix());
-            ActionSupport.buildEvent(profileRequestContext, IdPEventIds.INVALID_RELYING_PARTY_CTX);
-            return;
-        }
-        log.debug("Attaching RelyingPartyContext for {} and set it verified", entityId);
-        rpContext.setRelyingPartyId(entityId);
-        rpContext.setVerified(true);
     }
 
     /**

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


More information about the commits mailing list