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
F18756179: D18897.diff
Sun, Oct 5, 9:53 AM
F18699095: D18897.id.diff
Sat, Sep 27, 12:24 PM
F18682168: D18897.diff
Fri, Sep 26, 5:18 AM
F18650011: D18897.id45313.diff
Sat, Sep 20, 11:04 PM
F18600182: D18897.id45313.diff
Sep 13 2025, 9:09 AM
F18566460: D18897.id.diff
Sep 9 2025, 2:39 PM
F18501974: D18897.diff
Sep 4 2025, 10:05 PM
F18334233: D18897.diff
Aug 25 2025, 4:44 AM
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