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
F13128058: D18899.diff
Tue, Apr 30, 10:58 PM
Unknown Object (File)
Fri, Apr 19, 7:07 PM
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
View All 90 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
Branch
revoke9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19102
Build 25785: Run Core Tests
Build 25784: arc lint + arc unit

Revision Contents
Changeset List

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

Diff 45315

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