Page MenuHomePhabricator

Need to force users to reset their phabricator TOTP auth factor
Closed, ResolvedPublicSecurity

Description

For users who have multifactor auth set up prior to August of 2019, we need to force them to create a new auth factor / delete the old one.

To do this, the proposal is to redirect them to the settings page and provide instructions on how reset their second factor.

When an affected user logs in, I've modified the multifactor prompt to display a warning:

Screenshot from 2020-01-21 15-23-52.png (616×1 px, 46 KB)

After completing the TOTP prompt, the user will be redirected to the multifactor configuration settings UI, which should look something like this:

Screenshot from 2020-01-21 15-17-07.png (632×1 px, 88 KB)

Details

Author Affiliation
WMF Technology Dept

Event Timeline

Why do we need that, if I may ask?

@Urbanecm I suspect it's related to [[ https://lists.wikimedia.org/pipermail/wikitech-l/2020-January/092960.html | [Wikitech-l] 14 January 2020 security incident on Phabricator ]] & Blog Post: 14 January 2020 security incident on Phabricator.

@mmodell Is disabling and enabling MFA enough? If so, I can do it now instead of waiting for the workarounds, etc. Thanks.

@MarcoAurelio Yes, you can do it on your own manually. The best way seems to be to first create a third auth factor in Phab, then delete the old one in Phab. If you leave both there, you will have 3FA and have to use both until the old one is removed. And please don't remove anything from your phone's authenticator app until you after are done in Phab.

And yes, this is being done as a purely precautionary measure related to the Wikitech-l message referenced above and the associated blog post .

Ok so I've got everything ready to deploy. Should I request a dedicated deployment window for this?

I would suggest stepping around/over All Hands, in case anything goes wrong.

@Dsharpe I was planning to deploy this today along with a lot of other stuff that's been backing up in my undeployed queue. Do you really think it should wait for ~2 weeks?

Your planned schedule is far better. I just didn't want to rush anyone.

mmodell changed the visibility from "Custom Policy" to "Public (No Login Required)".Jan 23 2020, 4:53 PM
mmodell changed the edit policy from "Custom Policy" to "All Users".

Should the affected accounts be logged out? Phab sessions are long-lived so a login change might not have much immediate effect otherwise.