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
F19488439: D21016.diff
Thu, Jan 8, 5:08 AM
F19192733: D21016.id50061.diff
Tue, Dec 16, 10:21 AM
F19083178: D21016.id50079.diff
Dec 2 2025, 12:31 PM
F19083177: D21016.id50071.diff
Dec 2 2025, 12:31 PM
F19002340: D21016.id.diff
Nov 21 2025, 5:27 AM
F18985349: D21016.id50061.diff
Nov 17 2025, 12:59 PM
F18822141: D21016.id50061.diff
Oct 23 2025, 4:29 AM
F18808213: D21016.id.diff
Oct 19 2025, 5: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