necouchman commented on PR #915:
URL: https://github.com/apache/guacamole-client/pull/915#issuecomment-1749776979

   Uh-oh, I suspect that I've over-looked something very important, here, in 
the authentication flow with this module. Since the Duo module is just doing 
the 2FA verification and not actual authentication, I suspect that this is 
never going to work as-implemented - that is, the 
`DuoAuthenticationSessionManager` is never going to get referenced, because 
it's going to want the initial authentication (username + password) on the 
redirect back.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@guacamole.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to