-
Notifications
You must be signed in to change notification settings - Fork 260
chore(auth): add email otp mfa enums and types #5237
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
Changes from all commits
1b5d2bd
a6492d7
755f30d
95f658b
1f72aca
a5abb1e
edaf999
630adad
3f4efce
438f800
b4a8ceb
d7f3d5d
c98ce74
c1d33e3
18a5316
d484c55
f604447
b0c03b7
5153683
1d37cae
e69d96b
764f16d
26becec
86a3228
13e60c4
0d725dc
69abe75
970bf15
5586c16
4725074
f0b69a3
7b901af
66097f2
0fdcc15
e630a30
334e8b3
87bb72b
c4ee27d
dd76942
d602bef
e98a31c
d7ebd26
aad9883
9d9abe8
e7e0888
4135f06
eb93be4
daacad7
a3c57e9
7129360
865d204
2fddb4a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,7 +35,7 @@ void main() { | |
signInRes.nextStep.signInStep, | ||
because: 'MFA is required, and TOTP is chosen when ' | ||
'no phone number is registered', | ||
).equals(AuthSignInStep.continueSignInWithTotpSetup); | ||
).equals(AuthSignInStep.continueSignInWithMfaSetupSelection); | ||
khatruong2009 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
final sharedSecret = signInRes.nextStep.totpSetupDetails!.sharedSecret; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Q: is There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's correct, There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do you mean signInRes.nextStep.totpSetupDetails is not null for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes |
||
final setupRes = await Amplify.Auth.confirmSignIn( | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -34,7 +34,7 @@ void main() { | |
signInRes.nextStep.signInStep, | ||
because: | ||
"TOTP MFA is automatically enabled when it's the only option", | ||
).equals(AuthSignInStep.continueSignInWithTotpSetup); | ||
).equals(AuthSignInStep.continueSignInWithMfaSetupSelection); | ||
|
||
final sharedSecret = signInRes.nextStep.totpSetupDetails!.sharedSecret; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as above There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. answered here |
||
final setupRes = await Amplify.Auth.confirmSignIn( | ||
|
Uh oh!
There was an error while loading. Please reload this page.