Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding a method for exchanging 2FA token to the SAML response #44

Merged
merged 13 commits into from
Jul 26, 2024

Conversation

yevgenkre
Copy link
Contributor

The Epic integration uses the SAML response from the Duo server to handle the authentication on it's end. This change introduces the method to retrieve the SAML response in the end of the authentication in exchange to the 2FA token.

Description

Added a method ExchangeAuthorizationCodeForSamlResponse(string duoCode, string username) which sends the 2FA token to the Duo server, receives the response and returns the samlResponse part of it.

Motivation and Context

This change is a part of the bigger effort of migrating Epic integration to the Universal Prompt from the legacy implementation. For this reason the Universal Prompt needs to support the SAML response.

How Has This Been Tested?

Added unit tests, specific for SAML response. Made sure that the existing unit tests pass

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

DuoUniversal/Client.cs Outdated Show resolved Hide resolved
DuoUniversal/Client.cs Outdated Show resolved Hide resolved
@AaronAtDuo AaronAtDuo merged commit 4a5042a into duosecurity:main Jul 26, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants