Tags

None

Referenced Files

	F18741652: D18904.diff
	Thu, Oct 2, 2:57 PM

	F18728721: D18904.id45320.diff
	Tue, Sep 30, 9:45 AM

	F18654099: D18904.diff
	Mon, Sep 22, 2:37 AM

	F18605073: D18904.id45320.diff
	Sep 13 2025, 7:35 PM

	F18597907: D18904.id45364.diff
	Sep 13 2025, 3:55 AM

	F18591089: D18904.id.diff
	Sep 12 2025, 8:12 AM

	F18586327: D18904.diff
	Sep 11 2025, 6:18 PM

	F18509242: D18904.id.diff
	Sep 5 2025, 3:18 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13043: Improve authentication revocation behaviors

Commits

rPcab2bba6f2f7: Remove "passwordHash" and "passwordSalt" from User objects

Summary

Ref T13043. After D18903, this data has migrated to shared infrastructure and has no remaining readers or writers.

Just delete it now, since the cost of a mistake here is very small (users need to "Forgot Password?" and pick a new password).

Test Plan

Grepped for passwordHash, passwordSalt, and variations.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 22 2018, 1:20 AM

Harbormaster completed remote builds in B19107: Diff 45320.Jan 22 2018, 1:21 AM

epriestley requested review of this revision.Jan 22 2018, 1:21 AM

epriestley added a child revision: D18906: Add a rate limit for guessing old passwords when changing passwords.Jan 22 2018, 1:52 AM

amckinley accepted this revision.Jan 23 2018, 7:48 PM

This revision is now accepted and ready to land.Jan 23 2018, 7:48 PM

Closed by commit rPcab2bba6f2f7: Remove "passwordHash" and "passwordSalt" from User objects (authored by epriestley). · Explain WhyJan 23 2018, 9:44 PM

This revision was automatically updated to reflect the committed changes.

Remove "passwordHash" and "passwordSalt" from User objects
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 45364

resources/sql/autopatches/20180121.auth.05.accountnuke.sql

src/applications/people/storage/PhabricatorUser.php

Remove "passwordHash" and "passwordSalt" from User objectsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 45364

resources/sql/autopatches/20180121.auth.05.accountnuke.sql

src/applications/people/storage/PhabricatorUser.php

Remove "passwordHash" and "passwordSalt" from User objects
ClosedPublic
Actions

Revision Contents
Changeset List