HomePhorge

Internalize storage access for PhabricatorUserOAuthInfo

Description

Internalize storage access for PhabricatorUserOAuthInfo

Summary:
Ref T1536. Move all access to the underlying storage to inside the class. My plan is:

  • Migrate the table to ExternalAccount.
  • Nuke the table.
  • Make this class read from and write to ExternalAccount instead.

We can't get rid of OAuthInfo completely because Facebook still depends on it for now, via registration hooks.

Test Plan: Logged in and registered with OAuth.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T1536

Differential Revision: https://secure.phabricator.com/D6171

Details

Provenance
epriestleyAuthored on Jun 14 2013, 7:00 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP3005811b9e13: Remove OAuth token/expiry interfaces
Branches
Unknown
Tags
Unknown

Event Timeline