[java-identity-provider] branch main updated: Fix incorrect preExecute handling.

Scott Cantor cantor.2 at osu.edu
Wed Feb 9 17:27:24 UTC 2022


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=478309c10d73749ab0f6249235768af8061fcad9

The following commit(s) were added to refs/heads/main by this push:
     new 478309c10 Fix incorrect preExecute handling.
478309c10 is described below

commit 478309c10d73749ab0f6249235768af8061fcad9
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Wed Feb 9 12:27:21 2022 -0500

    Fix incorrect preExecute handling.
---
 .../idp/saml/saml2/profile/impl/PopulateEncryptionParameters.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/PopulateEncryptionParameters.java b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/PopulateEncryptionParameters.java
index 3240a0ae4..5b41d7d93 100644
--- a/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/PopulateEncryptionParameters.java
+++ b/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/PopulateEncryptionParameters.java
@@ -238,6 +238,10 @@ public class PopulateEncryptionParameters extends AbstractProfileAction {
     @Override
     protected boolean doPreExecute(@Nonnull final ProfileRequestContext profileRequestContext) {
         
+        if (!super.doPreExecute(profileRequestContext)) {
+            return false;
+        }
+        
         final RelyingPartyContext rpContext = relyingPartyContextLookupStrategy.apply(profileRequestContext);
         if (rpContext == null) {
             log.debug("{} Unable to locate RelyingPartyContext", getLogPrefix());
@@ -292,7 +296,7 @@ public class PopulateEncryptionParameters extends AbstractProfileAction {
         log.debug("{} Encryption for assertions ({}), identifiers ({}), attributes({})", getLogPrefix(),
                 encryptAssertions, encryptIdentifiers, encryptAttributes);
         
-        return super.doPreExecute(profileRequestContext);
+        return true;
     }
     
     /** {@inheritDoc} */

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


More information about the commits mailing list