Page MenuHomePhabricator

When users have no password on their account, guide them through the "reset password" flow in the guise of "set password"
ClosedPublic

Authored by epriestley on Feb 7 2019, 3:25 AM.
Tags
None
Referenced Files
F15552523: D20120.id48132.diff
Sun, Apr 27, 9:19 PM
F15552190: D20120.id48035.diff
Sun, Apr 27, 7:47 PM
F15546979: D20120.id.diff
Sat, Apr 26, 5:37 PM
F15541832: D20120.diff
Fri, Apr 25, 1:43 PM
F15531875: D20120.diff
Wed, Apr 23, 1:32 PM
F15526978: D20120.id.diff
Mon, Apr 21, 11:54 PM
F15447017: D20120.id48035.diff
Mar 27 2025, 9:26 PM
F15446822: D20120.id48132.diff
Mar 27 2025, 8:26 PM
Subscribers

Details

Summary

Depends on D20119. Fixes T9512. When you don't have a password on your account, the "Password" panel in Settings is non-obviously useless: you can't provide an old password, so you can't change your password.

The correct remedy is to "Forgot password?" and go through the password reset flow. However, we don't guide you to this and it isn't really self-evident.

Instead:

  • Guide users to the password reset flow.
  • Make it work when you're already logged in.
  • Skin it as a "set password" flow.

We're still requiring you to prove you own the email associated with your account. This is a pretty weak requirement, but maybe stops attackers who use the computer at the library after you do in some bizarre emergency and forget to log out? It would probably be fine to just let users "set password", this mostly just keeps us from having two different pieces of code responsible for setting passwords.

Test Plan
  • Set password as a logged-in user.
  • Reset password on the normal flow as a logged-out user.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable