[java-identity-provider] branch master updated: IDP-1387 - Config-based ForceAuthn enhancement broke renew flag for CAS

Scott Cantor cantor.2 at osu.edu
Fri Jan 4 18:59:38 EST 2019


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

scantor pushed a commit to branch master
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=7bea9bbceda33b35dd8b4fa510024422c0e1cda3

The following commit(s) were added to refs/heads/master by this push:
       new  7bea9bb   IDP-1387 - Config-based ForceAuthn enhancement broke renew flag for CAS
7bea9bb is described below

commit 7bea9bbceda33b35dd8b4fa510024422c0e1cda3
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Fri Jan 4 18:58:14 2019 -0500

    IDP-1387 - Config-based ForceAuthn enhancement broke renew flag for CAS
    
    https://issues.shibboleth.net/jira/browse/IDP-1387
---
 .../idp/cas/flow/impl/BuildAuthenticationContextAction.java       | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/BuildAuthenticationContextAction.java b/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/BuildAuthenticationContextAction.java
index 277031f..6f87a52 100644
--- a/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/BuildAuthenticationContextAction.java
+++ b/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/flow/impl/BuildAuthenticationContextAction.java
@@ -55,9 +55,11 @@ public class BuildAuthenticationContextAction extends
         ac.setForceAuthn(getCASRequest(profileRequestContext).isRenew());
         ac.setIsPassive(false);
 
-        final LoginConfiguration config = configLookupFunction.apply(profileRequestContext);
-        if (config != null) {
-            ac.setForceAuthn(config.getForceAuthnPredicate().apply(profileRequestContext));
+        if (!ac.isForceAuthn()) {
+            final LoginConfiguration config = configLookupFunction.apply(profileRequestContext);
+            if (config != null) {
+                ac.setForceAuthn(config.getForceAuthnPredicate().apply(profileRequestContext));
+            }
         }
         
         final AuthenticationContext initialAuthnContext =

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


More information about the commits mailing list