[java-identity-provider] branch main updated: Replace deprecated calls.

Scott Cantor cantor.2 at osu.edu
Thu Oct 1 12:25:24 UTC 2020


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=aed5a67a51c479314f504298657fbc97a5b684e1

The following commit(s) were added to refs/heads/main by this push:
       new  aed5a67a5 Replace deprecated calls.
aed5a67a5 is described below

commit aed5a67a51c479314f504298657fbc97a5b684e1
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu Oct 1 08:25:21 2020 -0400

    Replace deprecated calls.
---
 .../idp/authn/duo/impl/ValidateDuoAuthAPI.java           | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/duo/impl/ValidateDuoAuthAPI.java b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/duo/impl/ValidateDuoAuthAPI.java
index 95ee8aca2..4c2050744 100644
--- a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/duo/impl/ValidateDuoAuthAPI.java
+++ b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/duo/impl/ValidateDuoAuthAPI.java
@@ -204,13 +204,13 @@ public class ValidateDuoAuthAPI extends AbstractValidationAction {
             log.info("{} No DuoAuthenticationContext available", getLogPrefix());
             handleError(profileRequestContext, authenticationContext, "No DuoAuthenticationContext context available",
                     AuthnEventIds.INVALID_AUTHN_CTX);
-            recordFailure();
+            recordFailure(profileRequestContext);
             return false;
         } else if (duoContext.getFactor() == null) {
             log.info("{} No factor set in DuoAuthenticationContext", getLogPrefix());
             handleError(profileRequestContext, authenticationContext, "No Duo factor set in DuoAuthenticationContext",
                     AuthnEventIds.REQUEST_UNSUPPORTED);
-            recordFailure();
+            recordFailure(profileRequestContext);
             return false;
         }
 
@@ -239,7 +239,7 @@ public class ValidateDuoAuthAPI extends AbstractValidationAction {
             if (DuoAuthAPI.DUO_PREAUTH_RESULT_ALLOW.equals(preAuthResult)) {
                 // User in bypass mode; treat as authenticated.
                 log.info("{} Duo pre-authentication (bypass) succeeded for '{}'", getLogPrefix(), username);
-                recordSuccess();
+                recordSuccess(profileRequestContext);
                 buildAuthenticationResult(profileRequestContext, authenticationContext);
                 return;
             }
@@ -251,7 +251,7 @@ public class ValidateDuoAuthAPI extends AbstractValidationAction {
                 handleError(profileRequestContext, authenticationContext,
                         String.format("%s:%s:%s", preAuthResult, username, preAuthResponse.getStatusMessage()),
                         AuthnEventIds.ACCOUNT_ERROR);
-                recordFailure();
+                recordFailure(profileRequestContext);
                 return;
             }
             
@@ -275,7 +275,7 @@ public class ValidateDuoAuthAPI extends AbstractValidationAction {
                             username, duoContext.getDeviceID());
                     handleError(profileRequestContext, authenticationContext, AuthnEventIds.INVALID_CREDENTIALS,
                             AuthnEventIds.INVALID_CREDENTIALS);
-                    recordFailure();
+                    recordFailure(profileRequestContext);
                     return;
                 }
             }
@@ -291,20 +291,20 @@ public class ValidateDuoAuthAPI extends AbstractValidationAction {
             if (DuoAuthAPI.DUO_AUTH_RESULT_ALLOW.equals(authResult)) {
                 log.info("{} Duo authentication succeeded for '{}' (Factor: {}, Device: {})", getLogPrefix(), username,
                         duoContext.getFactor(), duoContext.getDeviceID());
-                recordSuccess();
+                recordSuccess(profileRequestContext);
                 buildAuthenticationResult(profileRequestContext, authenticationContext);
             } else if (DuoAuthAPI.DUO_AUTH_RESULT_DENY.equals(authResult)) {
                 log.info("{} Duo authentication failed for '{}'", getLogPrefix(), username);
                 handleError(profileRequestContext, authenticationContext, authenticationResponse.getStatus(),
                         AuthnEventIds.INVALID_CREDENTIALS);
-                recordFailure();
+                recordFailure(profileRequestContext);
             } else {
                 throw new DuoWebException("Unexpected authentication response");
             }
         } catch (final DuoWebException e) {
             log.error("{} Duo AuthAPI access failed for '{}'", getLogPrefix(), username, e);
             handleError(profileRequestContext, authenticationContext, e, AuthnEventIds.AUTHN_EXCEPTION);
-            recordFailure();
+            recordFailure(profileRequestContext);
         }
     }
     // CheckStyle: CyclomaticComplexity|MethodLength|ReturnCount OFF

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


More information about the commits mailing list