Allow ConduitClient to use OAuth access tokens for authentication
ClosedPublic
Actions

Authored by epriestley on Apr 5 2016, 10:00 PM.

Details

Reviewers

Maniphest Tasks

Commits

rPHUa6d22b5cc662: (stable) Promote 2016 Week 15
rPHU22a0253e08fd: Allow ConduitClient to use OAuth access tokens for authentication

Summary

Ref T7303. This adds OAuth token support to ConduitClient.

Test Plan

Logged out of an instance and got logged out of the linked upstream; see followup diffs.

Diff Detail

Repository

rPHU libphutil

Branch

oauth9

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 11528
Build 14393: Run Core Tests
Build 14392: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 37661.Apr 5 2016, 10:00 PM

epriestley retitled this revision from to Allow ConduitClient to use OAuth access tokens for authentication.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T7303: Provide OAuth access to Conduit.

chad accepted this revision.Apr 5 2016, 10:08 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 5 2016, 10:08 PM

epriestley added inline comments.Apr 5 2016, 10:11 PM

src/auth/PhutilPhabricatorAuthAdapter.php
66	(This isn't directly related, but this scope no longer exists so it stops the endpoint from complaining.)

Closed by commit rPHU22a0253e08fd: Allow ConduitClient to use OAuth access tokens for authentication (authored by epriestley, committed by epriestley). · Explain WhyApr 5 2016, 10:19 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

auth/

PhutilPhabricatorAuthAdapter.php

2 lines

conduit/

ConduitClient.php

10 lines

Diff 37661

View Options

src/auth/PhutilPhabricatorAuthAdapter.php