-
Notifications
You must be signed in to change notification settings - Fork 163
Stuck on revalidate now loop #629
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
Comments
I'm wondering if this should be the expected behaviour, and the email delivery should be fixed in order to regain access. Allowing any kind of override could be abused when the admins actually want to enforce the email second factor, for example. The plugin now recommends enabling the backup codes which could be used in these instances. |
I'd suggest that the issue in this case is that Email can be activated as a 2FA method without confirmation that the user can receive emails. Just like TOTP and Security keys require you to confirm with the device to set it up, email should too. |
The user email is assumed to be valid by WP core since it is also used for password resets and other critical notifications. I feel like it would also add unnecessary friction to the setup flow if we enforced email validation. I suggest we don't implement this. |
I have a similar problem. After first installtation options are greyed out and I am asked to revalidate. I then have to provide a TOTP code that does not yet exist. |
Seems to be a duplicate of (or at least related to) #572. |
I have a similar problem. Steps I took to reproduce the issue:
I now cannot disable 2FA or change authentication method on my account without disabling the entire plugin. I have used this plugin on multiple other sites with no issue, this is the first time I have seen this. The only difference between them is that the site with this issue is a multisite. As well, when I look in the database the sites that are working have various I am using WordPress version 6.8.1 |
Describe the bug
In cases where email doesn't work, the plugin is stuck in "Revalidate Now" mode. The solution may be to allow changing the authentication method without requiring validation if email is the primary and only method currently set up with the two-factor plugin.
Steps to Reproduce
Steps I took to reproduce the issue:
Screenshots, screen recording, code snippet
Environment information
Please confirm that you have searched existing issues in this repository.
Yes
Please confirm that you have tested with all plugins deactivated except Two-Factor.
Yes
The text was updated successfully, but these errors were encountered: