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

Request 2FA code fbefore disable / Seed for backup code #26

Open
renatodvc opened this issue Jun 18, 2018 · 2 comments
Open

Request 2FA code fbefore disable / Seed for backup code #26

renatodvc opened this issue Jun 18, 2018 · 2 comments
Assignees

Comments

@renatodvc
Copy link

First let me say I'm new at Github, so I'm sorry if this is not the right place to raise questions and suggestions.
Also, would like to thanks the devs, this is the best MFA package for django that I found so far. Really appreciate the work!

I would like to suggest that it would be a good security practice to ask for a 2FA code when the user choose to disable the MFA Auth. [Prevent someone with physical acess to the PC from disabling it, while session is still valid.

I also think it would be a good idea to provide the key together with the QR Code, at the configure.html, so the user can print/copy/write it as a backup code. I have tried to do it, unsuccefully so far. I belive that's because I'm not familiar with the encode and decode funcs in the configure_mfa at views.py. If you don't intent to add this feature, I would appreciate if someone could shed some light on how can i do it myself.

Thanks,

@GuganVignesh
Copy link

Hello team,

Well done so far, but ...

Yeah! thanks @renatodvc for putting it forth, definitely these are one of the most important features that need implementations that are needed for MFA to be called as 'Stable Version'.

Yes this in-turn adds a curious two important requirements so far,

  1. Ask for the code while disabling MFA
  2. Providing backup codes

Please it would be great if we have these two in-place as soon as possible, so that it will eliminate all the other necessary temporary changes that are needed to be done in our applications.

Please do consider this as a priority features for the next upgrade.

Thanks,

@GuganVignesh
Copy link

@chaitu210 please do consider this as an important feature for the next upgrade of this package. Thanks,

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

No branches or pull requests

3 participants