-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add GetUserData to obtain PreferredMfaSetting #1204
Comments
I'm really confused at how this is supposed to work... We've got 2 flows: PATH 1: when we run PATH 1
PATH 2:
Surely both should follow the same response format? they are currently completely different. What adds even more complexity to this is if I now set SMS after I've enabled TOTP, I get the following response:
So now we have 1 flow for SMS, 1 flow for TOTP... 2 completely different responses.. If I overwrite the TOTP path with SMS.. I get the correct response back for both flows.. but not the initial SMS setup (PATH1)..
#1259 raised here |
@MiroslavPetrik I agree. I created a new api for developers to get the current preferred mfa type: #1254 |
This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs. Looking for a help forum? We recommend joining the Amplify Community Discord server |
Do you want to request a feature or report a bug?
feature
What is the current behavior?
When MFA is enabled with TOTP, the
getMFAOptions
returnsundefined
.#360
However I need to know whether the MFA is enabled... I am able to get
PreferredMfaSetting: "SOFTWARE_TOKEN_MFA"
from a call tocognitoUser.getUserData
https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_GetUser.html
or use case 31 in
https://github.com/aws/aws-amplify/tree/master/packages/amazon-cognito-identity-js
but there is no respective method in amplify so would be nice to have it, or enhance the
getMFAOptions
.What is the expected behavior?
getMFAOptions returns info about TOTP.
The text was updated successfully, but these errors were encountered: