-
-
Notifications
You must be signed in to change notification settings - Fork 926
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
MFA Device Replacement Flow #3142
Comments
@simi @sonalkr132 If you have time, could you have a look at this? It would be a "nice to have" if, when mfa is required, people could change their mfa device. |
@kevinlinxc seems ok to me 💪. Feel free to open PR so we can do review in here. |
what happens if user loses their mfa device and recovery keys? As of now, we disable their mfa manually using disable_mfa script. I am guessing the script will continue to work. I am hoping mfa required users will be able to register new device if their mfa is disabled. |
@jenshenny is there still plan to contribute this feature? |
It's still on my radar. With the introduction of security devices, the flow would probably need to be changed. If a MFA req user tries to disable their last MFA device, they should be prompted to setup a TOTP or security device. That being said, this is my mental model of how the MFA settings should look like (I can chip away at it).
|
Currently, if a user wants to change MFA devices, they have to disable MFA and add a new MFA device.
In the future, when MFA for a user cannot be disabled (for users with required mfa), disabling->enabling won't be an option, as there would necessarily be a state where the user has disabled MFA.
I've collaborated with some people to make a prototype to address this, and it boils down to:
Demonstration:
https://user-images.githubusercontent.com/44324811/178588968-b34b02b0-d672-4dad-aa65-35e95e572a70.mp4
(The first and last part are showing that you can only access the replacement page with verification)
Prototype code is here: Shopify#22
Feedback is welcome, especially vis-à-vis the placement/look of the starting button as that seems a bit unnatural right now.
The text was updated successfully, but these errors were encountered: