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

Second authentication/confirmation to disable 2FA #214

Open
stronenv opened this issue Jun 8, 2023 · 6 comments
Open

Second authentication/confirmation to disable 2FA #214

stronenv opened this issue Jun 8, 2023 · 6 comments
Labels
enhancement New feature or request ui Related to user interface

Comments

@stronenv
Copy link

stronenv commented Jun 8, 2023

When disabling 2FA, you're not asked to confirm by entering a 2FA code, password, or asked to confirm.

Suggested improvements for disabling 2FA:

  1. Second confirmation with a "Are you sure?" prompt
  2. Second confirmation by entering password or 2FA code (or backup codes if you don't have access to your device.)
@pkevan
Copy link
Contributor

pkevan commented Jun 8, 2023

Thanks for the report @stronenv, when you disabled 2FA was this shortly after it being enabled?

@stronenv
Copy link
Author

stronenv commented Jun 8, 2023

Hi @pkevan! Yes, it was shortly after. I did disable it on another device, though, after logging out and back in again.

@pkevan
Copy link
Contributor

pkevan commented Jun 8, 2023

Thanks for the extra details - we'll investigate further.

@iandunn iandunn added this to the Iteration 1 milestone Jun 8, 2023
@renintw
Copy link
Contributor

renintw commented Jun 8, 2023

@iandunn Do you think it makes sense if we implement the first suggestion Second confirmation with a "Are you sure?" prompt first and leave the second one in iteration 2?

@renintw renintw added enhancement New feature or request ui Related to user interface labels Jun 8, 2023
@iandunn
Copy link
Member

iandunn commented Jun 9, 2023

a "Are you sure?" prompt

That seems prudent 👍🏻 , but not necessarily high priority IMO, since it's easy to turn back on, and there are several status indicators to make it obvious when it's off.

password or 2FA code

🤔 We already have this in the revalidation process. It sounds like it maybe wasn't triggered in this case because of the time window where it's not required (ala sudo mode).

If not, then that seems like it'd indicate a bug w/ the existing code that should be fixed instead.

Maybe I missed something though?

Possibly related WordPress/two-factor#578

@pkevan
Copy link
Contributor

pkevan commented Jun 10, 2023

🤔 We already have this in the revalidation process. It sounds like it maybe wasn't triggered in this case because of the time window where it's not required (ala sudo mode).

Yes, I think this might be the case in the 2nd device, since the time window would start from log in.

@iandunn iandunn modified the milestones: Iteration 1, Iteration 2 Sep 13, 2023
@StevenDufresne StevenDufresne removed this from the Iteration 2 milestone Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ui Related to user interface
Projects
None yet
Development

No branches or pull requests

5 participants