Page MenuHomePhabricator

Require application "Can Use" capability to call Conduit methods
ClosedPublic

Authored by epriestley on Oct 17 2013, 7:47 PM.
Tags
None
Referenced Files
F13083427: D7342.diff
Wed, Apr 24, 10:32 PM
Unknown Object (File)
Fri, Apr 19, 3:26 AM
Unknown Object (File)
Fri, Apr 19, 3:26 AM
Unknown Object (File)
Fri, Apr 19, 3:26 AM
Unknown Object (File)
Wed, Apr 17, 6:43 AM
Unknown Object (File)
Thu, Apr 11, 4:37 AM
Unknown Object (File)
Tue, Apr 9, 8:55 AM
Unknown Object (File)
Mon, Apr 8, 3:30 AM
Subscribers

Details

Summary

Ref T603. If you don't have access to an application, prevent execution of its (authenticated) methods.

Test Plan

Restricted Tokens to only admins, then tried to view/call Token methods as a non-admin.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

I want to clean up Conduit auth at some point -- and maybe sooner than later -- but there are a few related changes (the SSH version, making a more cURL-able version of the API available, formalizing the non-connect version, typechecking, getting rid of differential.anonymous-access, cleaning up the way tokens cycle and we deal with out-of-range client timestmaps) that probably make sense to bundle most of together so all the testing can overlap. That should let us run more of these checks through less total code.