Page MenuHomePhabricator
Differential D21016

Write ExternalAccountIdentifiers when interacting with external authentication providers
ClosedPublic
Actions

Authored by epriestley on Feb 22 2020, 4:30 PM.
Tags
None
Referenced Files
F19083178: D21016.id50079.diff
Tue, Dec 2, 12:31 PM
F19083177: D21016.id50071.diff
Tue, Dec 2, 12:31 PM
F19002340: D21016.id.diff
Fri, Nov 21, 5:27 AM
F18985349: D21016.id50061.diff
Mon, Nov 17, 12:59 PM
F18822141: D21016.id50061.diff
Oct 23 2025, 4:29 AM
F18808213: D21016.id.diff
Oct 19 2025, 5:27 AM
F18803780: D21016.diff
Oct 18 2025, 3:56 AM
F18778194: D21016.id.diff
Oct 11 2025, 8:27 AM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rPbcaf60015a47: Write ExternalAccountIdentifiers when interacting with external authentication…
Summary

Depends on D21015. When we sync an external account and get a list of account identifiers, write them to the database.

Nothing reads them yet and we still write "accountId", this just prepares us for reads.

Test Plan

Linked, refreshed, unlinked, and re-linked an external account. Peeked at the database and saw a sensible-looking row.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 50079

View Options

src/applications/auth/controller/PhabricatorAuthUnlinkController.php

Loading...
View Options

src/applications/auth/provider/PhabricatorAuthProvider.php

Loading...
View Options

src/applications/auth/query/PhabricatorExternalAccountQuery.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccount.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccountIdentifier.php

Loading...
Phabricator · Built by Phacility