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

Reauth 2nd factor to change 2FA settings #43

Closed
iandunn opened this issue Jan 23, 2023 · 3 comments · Fixed by #147
Closed

Reauth 2nd factor to change 2FA settings #43

iandunn opened this issue Jan 23, 2023 · 3 comments · Fixed by #147

Comments

@iandunn
Copy link
Member

iandunn commented Jan 23, 2023

There's an upstream issue for this, but we'll probably need to do our own since we have the custom UI.

@iandunn iandunn added this to the Iteration 1 milestone Jan 23, 2023
@dd32
Copy link
Member

dd32 commented Feb 20, 2023

I've started work upstream in WordPress/two-factor#529 but there's a major catch - I've used WordPress sessions upstream, and WordPress.org doesn't support those.

In some brief testing, I believe that we can likely use WordPress Sessions on WordPress.org as well, there's some hacky-hackery that we could use to make WordPress see sessions, but have Trac/Codex use non-sessioned cookies. Either that, or just fix those to also use sessions finally.. I've done some work on Trac support already.

@StevenDufresne
Copy link
Contributor

Can we narrow down the focus of this ticket? Maybe generated a specific to-do list?

@dd32
Copy link
Member

dd32 commented Apr 5, 2023

@StevenDufresne Sure

  • Reauth 2FA tokens when changing 2FA details.
    • Prompt user if it's been >xx mins since last 2FA challenge before displaying UI.

Dependencies:

The dependencies are completed, but need testing, merging and/or systems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants