[java-identity-provider] branch main updated: IDP-2289 - Misnamed setter in AbstractSAMLProfileConfiguration

[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=d482a8dcc4e730dd3270002ba96612e5ec5b8ab5

The following commit(s) were added to refs/heads/main by this push:
     new d482a8dcc IDP-2289 - Misnamed setter in AbstractSAMLProfileConfiguration
d482a8dcc is described below

commit d482a8dcc4e730dd3270002ba96612e5ec5b8ab5
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu May 23 08:42:30 2024 -0400

    IDP-2289 - Misnamed setter in AbstractSAMLProfileConfiguration
    
    https://shibboleth.atlassian.net/browse/IDP-2289
---
 .../impl/AbstractSAMLProfileConfiguration.java     | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/config/impl/AbstractSAMLProfileConfiguration.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/config/impl/AbstractSAMLProfileConfiguration.java
index 83efb0891..1e2253526 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/config/impl/AbstractSAMLProfileConfiguration.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/profile/config/impl/AbstractSAMLProfileConfiguration.java
@@ -26,6 +26,8 @@ import net.shibboleth.shared.annotation.constraint.NotEmpty;
 import net.shibboleth.shared.logic.Constraint;
 import net.shibboleth.shared.logic.FunctionSupport;
 import net.shibboleth.shared.logic.PredicateSupport;
+import net.shibboleth.shared.primitive.DeprecationSupport;
+import net.shibboleth.shared.primitive.DeprecationSupport.ObjectType;
 
 import org.opensaml.messaging.context.MessageContext;
 import org.opensaml.profile.context.ProfileRequestContext;
@@ -112,7 +114,7 @@ public abstract class AbstractSAMLProfileConfiguration extends AbstractIntercept
     public Function<MessageContext,Exception> getMessageHandler(@Nullable final MessageContext messageContext) {
         return messageHandlerLookupStrategy.apply(messageContext);
     }
-    
+
     /**
      * Set a handler for the SAML message.
      * 
@@ -120,9 +122,25 @@ public abstract class AbstractSAMLProfileConfiguration extends AbstractIntercept
      * 
      * @since 5.0.0
      */
-    public void setMessageDecorator(@Nullable final Function<MessageContext,Exception> handler) {
+    public void setMessageHandler(@Nullable final Function<MessageContext,Exception> handler) {
         messageHandlerLookupStrategy = FunctionSupport.constant(handler);
     }
+
+    /**
+     * Set a handler for the SAML message.
+     * 
+     * @param handler message handler
+     * 
+     * @since 5.0.0
+     * 
+     * @deprecated
+     */
+    @Deprecated(forRemoval=true, since="5.0.0")
+    public void setMessageDecorator(@Nullable final Function<MessageContext,Exception> handler) {
+        DeprecationSupport.warn(ObjectType.METHOD, "SAMLProfileConfiguration.setMessageDecorator", "relying-party.xml",
+                "SAMLProfileConfiguration.setMessageHandler");
+        setMessageHandler(handler);
+    }
     
     /**
      * Set a lookup strategy for the handler for the SAML message.

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