Page MenuHomePhabricator

OAuth - add concept of "trusted" clients that get auto redirects
ClosedPublic

Authored by btrahan on Feb 9 2015, 10:16 PM.
Tags
None
Referenced Files
F18504864: D11724.id.diff
Fri, Sep 5, 12:07 AM
F18496532: D11724.diff
Thu, Sep 4, 5:30 PM
F18373167: D11724.id.diff
Thu, Aug 28, 7:13 AM
F18369885: D11724.id28232.diff
Thu, Aug 28, 2:52 AM
F18369867: D11724.id28231.diff
Thu, Aug 28, 2:50 AM
F18369768: D11724.id28232.diff
Thu, Aug 28, 2:40 AM
F18369723: D11724.id.diff
Thu, Aug 28, 2:36 AM
F18369703: D11724.id28231.diff
Thu, Aug 28, 2:34 AM
Subscribers

Details

Summary

Fixes T7153.

Test Plan

used bin/auth trust-oauth-client and bin/auth untrust-oauth-client to set the bit and verify error states.

registered via oauth with bin/auth trust-oauth-client set and I did not have the confirmation screen
registered via oauth with bin/auth untrust-oauth-client set and I did have the confirmation screen

Diff Detail

Repository
rP Phabricator
Branch
T7153_3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4411
Build 4424: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to OAuth - add concept of "trusted" clients that get auto redirects.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

Code looks solid to me.

This revision is now accepted and ready to land.Feb 9 2015, 10:18 PM
btrahan edited edge metadata.
This revision was automatically updated to reflect the committed changes.