[java-identity-provider] branch main updated: AttributeConsumingServiceIndex setting should be Integer.

Scott Cantor cantor.2 at osu.edu
Wed Feb 22 20:09:24 UTC 2023


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=02ef8b63876ca1a5dac0d0afa1d6c1fa9e00cdbe

The following commit(s) were added to refs/heads/main by this push:
     new 02ef8b638 AttributeConsumingServiceIndex setting should be Integer.
02ef8b638 is described below

commit 02ef8b63876ca1a5dac0d0afa1d6c1fa9e00cdbe
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Wed Feb 22 15:09:21 2023 -0500

    AttributeConsumingServiceIndex setting should be Integer.
---
 .../resources/net/shibboleth/idp/conf/relying-party-mddriven.xml  | 2 +-
 .../saml2/profile/config/impl/BrowserSSOProfileConfiguration.java | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/idp-conf-impl/src/main/resources/net/shibboleth/idp/conf/relying-party-mddriven.xml b/idp-conf-impl/src/main/resources/net/shibboleth/idp/conf/relying-party-mddriven.xml
index 690667eb5..c82330f65 100644
--- a/idp-conf-impl/src/main/resources/net/shibboleth/idp/conf/relying-party-mddriven.xml
+++ b/idp-conf-impl/src/main/resources/net/shibboleth/idp/conf/relying-party-mddriven.xml
@@ -542,7 +542,7 @@
             <bean parent="shibboleth.MDDrivenStringProperty" p:propertyName="spNameQualifier" />
         </property>
         <property name="attributeIndexLookupStrategy">
-            <bean parent="shibboleth.MDDrivenStringProperty" p:propertyName="attributeIndex" />
+            <bean parent="shibboleth.MDDrivenIntProperty" p:propertyName="attributeIndex" />
         </property>
     </bean>
 
diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/config/impl/BrowserSSOProfileConfiguration.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/config/impl/BrowserSSOProfileConfiguration.java
index 7da3c0ffc..a04fbf72b 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/config/impl/BrowserSSOProfileConfiguration.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/config/impl/BrowserSSOProfileConfiguration.java
@@ -139,7 +139,7 @@ public class BrowserSSOProfileConfiguration extends AbstractSAML2AssertionProduc
     @Nonnull private Function<ProfileRequestContext,String> spNameQualifierLookupStrategy;
 
     /** Lookup function to supply AttributeConsumingServiceIndex in request. */
-    @Nonnull private Function<ProfileRequestContext,String> attributeIndexLookupStrategy;
+    @Nonnull private Function<ProfileRequestContext,Integer> attributeIndexLookupStrategy;
     
     /** Lookup function to supply RequestedAttributes in request. */
     @Nonnull private Function<ProfileRequestContext,Collection<RequestedAttribute>> requestedAttributesLookupStrategy;
@@ -956,7 +956,7 @@ public class BrowserSSOProfileConfiguration extends AbstractSAML2AssertionProduc
     }
 
     /** {@inheritDoc} */
-    @Nullable public String getAttributeIndex(@Nullable final ProfileRequestContext profileRequestContext) {
+    @Nullable public Integer getAttributeIndex(@Nullable final ProfileRequestContext profileRequestContext) {
         return attributeIndexLookupStrategy.apply(profileRequestContext);
     }
 
@@ -967,7 +967,7 @@ public class BrowserSSOProfileConfiguration extends AbstractSAML2AssertionProduc
      * 
      * @since 5.0.0
      */
-    public void setAttributeIndex(@Nullable final String index) {
+    public void setAttributeIndex(@Nullable final Integer index) {
         attributeIndexLookupStrategy = FunctionSupport.constant(index);
     }
     
@@ -978,7 +978,7 @@ public class BrowserSSOProfileConfiguration extends AbstractSAML2AssertionProduc
      * 
      * @since 5.0.0
      */
-    public void setAttributeIndexLookupStrategy(@Nonnull final Function<ProfileRequestContext,String> strategy) {
+    public void setAttributeIndexLookupStrategy(@Nonnull final Function<ProfileRequestContext,Integer> strategy) {
         attributeIndexLookupStrategy = Constraint.isNotNull(strategy, "Lookup strategy cannot be null");
     }
 

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


More information about the commits mailing list