Page MenuHomePhabricator

Only let users log in to an OAuth server if they can see it
ClosedPublic

Authored by epriestley on Feb 5 2015, 6:55 PM.
Tags
None
Referenced Files
F14056285: D11696.diff
Sat, Nov 16, 7:35 PM
F14010682: D11696.diff
Thu, Oct 31, 11:37 AM
F14002323: D11696.id.diff
Fri, Oct 25, 5:53 PM
F14001550: D11696.id28142.diff
Fri, Oct 25, 9:20 AM
F13997368: D11696.diff
Thu, Oct 24, 4:03 AM
F13984884: D11696.id28143.diff
Oct 20 2024, 4:39 PM
Unknown Object (File)
Oct 11 2024, 1:10 AM
Unknown Object (File)
Sep 23 2024, 10:53 PM
Subscribers

Details

Summary

Fixes T7169. We just weren't doing a policy-aware query. Basic idea here is that if you set an app to be visible only to specific users, those specific users are the only ones who should be able to authorize it.

In the Phacility cluster, this allows us to prevent users who haven't been invited from logging in to an instance.

Test Plan
  • Tried to log into an instance I was not a member of.
  • Logged into an instance I am a member of.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Only let users log in to an OAuth server if they can see it.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 5 2015, 6:57 PM
This revision was automatically updated to reflect the committed changes.