Page MenuHomePhabricator

Don't prompt to upgrade unset passwords
ClosedPublic

Authored by epriestley on Feb 20 2014, 3:36 PM.
Tags
None
Referenced Files
F15449710: D8282.id19706.diff
Fri, Mar 28, 11:48 AM
F15444201: D8282.id.diff
Thu, Mar 27, 8:25 AM
F15443589: D8282.id19705.diff
Thu, Mar 27, 5:11 AM
F15439707: D8282.diff
Wed, Mar 26, 8:59 AM
F15435117: D8282.id.diff
Tue, Mar 25, 6:52 AM
F15433809: D8282.id19706.diff
Tue, Mar 25, 12:42 AM
F15429899: D8282.id19705.diff
Mon, Mar 24, 4:47 AM
F15427773: D8282.id19705.diff
Sun, Mar 23, 4:00 PM
Subscribers

Details

Summary

Fixes T4463. When your VCS or account password is not set, we test it for upgrade anyway. This doesn't make sense and throws shortly into the process because the empty hash isn't parseable.

Instead, only show upgrade prompts when the password exists.

Test Plan
  • Added a password to an existing account with no password via password reset.
  • Added a VCS password to an existing account with no VCS password.
  • Observed no fatals / nonsense behaviors.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped