[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