[java-idp-plugin-duo] branch main updated: Improve Duo controller error response

Phil Smart philip.smart at jisc.ac.uk
Thu Mar 11 11:43:23 UTC 2021


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

philsmart pushed a commit to branch main
in repository java-idp-plugin-duo.

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-plugin-duo.git;a=commit;h=4513c336b21ade3af058ded4b19de4ab7b68e813

The following commit(s) were added to refs/heads/main by this push:
       new  4513c33   Improve Duo controller error response
4513c33 is described below

commit 4513c336b21ade3af058ded4b19de4ab7b68e813
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Thu Mar 11 11:43:20 2021 +0000

    Improve Duo controller error response
---
 .../shibboleth/idp/plugin/authn/duo/impl/DuoOIDCAuthnController.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoOIDCAuthnController.java b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoOIDCAuthnController.java
index 7e81cd9..c322256 100644
--- a/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoOIDCAuthnController.java
+++ b/idp-duo-impl/src/main/java/net/shibboleth/idp/plugin/authn/duo/impl/DuoOIDCAuthnController.java
@@ -170,7 +170,8 @@ public class DuoOIDCAuthnController extends AbstractInitializableComponent{
             httpResponse.sendRedirect(authURL);
 
         } catch (final DuoClientException e) {
-            httpRequest.setAttribute(ExternalAuthentication.AUTHENTICATION_EXCEPTION_KEY, e);
+            log.error("Could not perform Duo 2FA authorization request",e);
+            httpRequest.setAttribute(ExternalAuthentication.AUTHENTICATION_ERROR_KEY, EventIds.INVALID_PROFILE_CTX);
             ExternalAuthentication.finishExternalAuthentication(key, httpRequest, httpResponse);
         }
               

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


More information about the commits mailing list