Page MenuHomePhabricator

Auth - restore Phabricator OAuth as a provider
ClosedPublic

Authored by btrahan on Jan 8 2015, 11:02 PM.
Tags
None
Referenced Files
F13303309: D11287.diff
Sat, Jun 8, 6:34 AM
F13286390: D11287.diff
Tue, Jun 4, 7:08 AM
F13273307: D11287.diff
Fri, May 31, 1:47 AM
F13238041: D11287.id27133.diff
Tue, May 21, 4:41 PM
F13203736: D11287.diff
May 15 2024, 12:05 AM
F13191916: D11287.id27120.diff
May 12 2024, 2:25 AM
F13191909: D11287.id27133.diff
May 12 2024, 2:20 AM
F13186264: D11287.diff
May 11 2024, 3:31 AM
Subscribers

Details

Reviewers
epriestley
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP2be746fb1f3a: Auth - restore Phabricator OAuth as a provider
Summary

So meta it hurts. Fixes T887.

Test Plan

created a second instance of phabricator locally. made an account on oauth server phabricator. set up my normal dev phabricator to use this new oauth phabricator. noted the form worked. created an account via the oauth method and it worked.

Diff Detail

Repository
rP Phabricator
Branch
T887
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3700
Build 3711: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan added a task: Restricted Maniphest Task.
btrahan retitled this revision from to Auth - restore Phabricator OAuth as a provider.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/auth/provider/PhabricatorPhabricatorAuthProvider.php
15

I think there's a way to do this in two stages, but we can clean that up in the long run. I think this is reasonably clear.

This revision is now accepted and ready to land.Jan 8 2015, 11:50 PM
btrahan edited edge metadata.

Make things two steps and re-jigger hint text a bit. I think its betterish now.

This revision was automatically updated to reflect the committed changes.