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

feat(auth): Resend MFA code #3409

Closed
wants to merge 16 commits into from
Closed

Conversation

dnys1
Copy link
Contributor

@dnys1 dnys1 commented Jul 12, 2023

  • Adds a resendSignInCode API to resend SMS MFA codes during the sign-in flow.
  • Adds a "Resend Code" button to the Authenticator for resending MFA codes during the sign-in flow.

@dnys1 dnys1 requested a review from a team as a code owner July 12, 2023 16:55
Dillon Nys added 13 commits July 17, 2023 09:06
Expands E2E backends for all TOTP/SMS MFA combinations.
Fix casing when mapping SDK exceptions
Implements TOTP MFA as part of the `signIn` flow. Adds category methods for setting up TOTP outside the `signIn` flow and for getting/setting the user's MFA preferences at any time.
Adds E2E tests for TOTP MFA features.
Due to a bug in URI, we need to manually encode query components so that the path encoding matches the query encoding. This allows Authenticator apps to correctly match the appName and issuer.
The slimmer client interface was not working when deploying from some systems.
Fixes issue on Windows with checking the `.dart_tool` directory by always running `pub get`.
Dillon Nys added 3 commits July 17, 2023 09:41
Adds a `resendSignInCode` API to resend SMS MFA codes during the sign-in flow.
Adds a `Resend Code` button to the sign-in SMS MFA confirmation screen.
@dnys1 dnys1 force-pushed the feat/auth/resend-mfa-code branch from f98c6d6 to 71c6c76 Compare July 19, 2023 15:26
@dnys1 dnys1 added pr:rebase PR should be submitted with a rebase commit pr:merge PR should be submitted with a merge commit and removed pr:rebase PR should be submitted with a rebase commit labels Jul 20, 2023
@dnys1 dnys1 closed this Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:merge PR should be submitted with a merge commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant