Page MenuHomePhabricator
Differential D18899

Remove the "PhabricatorRepositoryVCSPassword" class and table
ClosedPublic
Actions

Authored by epriestley on Jan 21 2018, 4:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 10, 9:36 PM
Unknown Object (File)
Wed, Apr 3, 4:01 PM
Unknown Object (File)
Tue, Apr 2, 4:47 PM
Unknown Object (File)
Tue, Apr 2, 4:47 PM
Unknown Object (File)
Tue, Apr 2, 4:47 PM
Unknown Object (File)
Tue, Apr 2, 4:47 PM
Unknown Object (File)
Tue, Mar 26, 9:12 PM
Unknown Object (File)
Feb 26 2024, 4:08 PM
View All 88 Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13043: Improve authentication revocation behaviors
Commits
rP753c4c5ff1ad: Remove the "PhabricatorRepositoryVCSPassword" class and table
Summary

Ref T13043. After D18898, this has been migrated to new, more modern storage and no longer has any readers or writers.

One migration from long ago (early 2014) is affected. Since this is ancient and the cost of dropping this is small (see inline), I just dropped it.

I'll note this in the changelog.

Test Plan

Ran migrations, got a clean bill of health from storage status. Grepped for removed symbol.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
32 lines
1 line
src/
2 lines
applications/
repository/
storage/

Diff 45355

View Options

resources/sql/autopatches/20140218.passwords.4.vcs.php

Loading...
View Options

resources/sql/autopatches/20180121.auth.01.vcsnuke.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryVCSPassword.php

Loading...
Phabricator · Built by Phacility