Page MenuHomePhabricator

Add test coverage to the PasswordEngine upgrade workflow and fix a few bugs
ClosedPublic

Authored by epriestley on Jan 21 2018, 3:44 PM.
Tags
None
Referenced Files
F19940623: D18897.diff
Wed, Apr 15, 3:40 AM
F19927023: D18897.id45313.diff
Tue, Apr 7, 7:36 PM
F19926578: D18897.id45312.diff
Tue, Apr 7, 5:09 PM
F19913489: D18897.diff
Fri, Apr 3, 3:18 AM
F19909945: D18897.id.diff
Thu, Apr 2, 3:35 AM
F19904709: D18897.id.diff
Tue, Mar 31, 11:34 AM
F19897741: D18897.diff
Sun, Mar 29, 8:50 AM
F19825952: D18897.diff
Mar 8 2026, 4:05 PM
Subscribers
None

Details

Summary

Ref T13043. When we verify a password and a better hasher is available, we automatically upgrade the stored hash to the stronger hasher.

Add test coverage for this workflow and fix a few bugs and issues, mostly related to shuffling the old hasher name into the transaction.

This doesn't touch anything user-visible yet.

Test Plan

Ran unit tests.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable