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

Support OOB MFA challenges #649

Merged
merged 4 commits into from
Feb 19, 2021
Merged

Support OOB MFA challenges #649

merged 4 commits into from
Feb 19, 2021

Conversation

ejensen
Copy link
Contributor

@ejensen ejensen commented Jan 28, 2021

Changes

This PR adds support for OOB MFA challenges by calling the /mfa/challenge endpoint when the MFA screen is shown. It uses the correct grant_type for the type of MFA challenge that was triggered. Previously, only the OTP MFA authenticator was supported. Now SMS/Voice/Email can also be used.

SMS_MFA

References

Testing

  • This change adds unit test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

@ejensen ejensen requested a review from a team as a code owner January 28, 2021 06:16
@Widcket Widcket added the needs investigation An issue that has more questions to answer or otherwise needs work to fully understand the issue label Feb 1, 2021
@ejensen ejensen marked this pull request as draft February 10, 2021 02:01
@ejensen ejensen force-pushed the mfa-support branch 2 times, most recently from fea7ed0 to 0509e0f Compare February 11, 2021 17:18
@ejensen ejensen changed the title [WIP] Support OOB MFA challenges Support OOB MFA challenges Feb 11, 2021
@Widcket
Copy link
Contributor

Widcket commented Feb 12, 2021

@ejensen Auth0.swift 1.31.0 is out, this can be updated now.

@ejensen ejensen force-pushed the mfa-support branch 2 times, most recently from 188935f to 7daf6b7 Compare February 16, 2021 23:45
@ejensen ejensen marked this pull request as ready for review February 16, 2021 23:45
Lock.podspec Outdated Show resolved Hide resolved
Copy link
Contributor

@Widcket Widcket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add tests.

@ejensen
Copy link
Contributor Author

ejensen commented Feb 18, 2021

@Widcket I added unit tests. I also enabled code coverage reporting within Xcode so we don't have to wait for CodeCov process to report

@Widcket
Copy link
Contributor

Widcket commented Feb 19, 2021

@Widcket I added unit tests. I also enabled code coverage reporting within Xcode so we don't have to wait for CodeCov process to report

The code coverage is already being enabled when running in CI.

@ejensen
Copy link
Contributor Author

ejensen commented Feb 19, 2021

Enabling code coverage within Xcode allows contributors to see both inline coverage and the overall coverage without waiting for the CodeCov CI job which only appears to run when the commit is made by an Auth0 team member.

Xcode coverage report

Inline code coverage

@Widcket Widcket merged commit d592da9 into auth0:master Feb 19, 2021
@Widcket Widcket deleted the mfa-support branch February 19, 2021 02:13
@Widcket
Copy link
Contributor

Widcket commented Feb 19, 2021

Since enabling code coverage within Xcode requires to modify the CI script, I'll raise that in a new PR.

@Widcket Widcket mentioned this pull request Feb 19, 2021
5 tasks
@Widcket Widcket added this to the vNext milestone Feb 19, 2021
@Widcket Widcket added CH: Added and removed needs investigation An issue that has more questions to answer or otherwise needs work to fully understand the issue labels Mar 9, 2021
@Widcket Widcket mentioned this pull request Mar 9, 2021
@Widcket
Copy link
Contributor

Widcket commented Mar 10, 2021

This is now out in v2.22.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants