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
F14494860: D7342.diff
Fri, Jan 3, 6:46 AM
Unknown Object (File)
Tue, Dec 31, 11:08 AM
Unknown Object (File)
Fri, Dec 20, 8:22 PM
Unknown Object (File)
Sun, Dec 15, 4:35 PM
Unknown Object (File)
Sat, Dec 14, 8:19 AM
Unknown Object (File)
Fri, Dec 13, 8:49 PM
Unknown Object (File)
Wed, Dec 11, 5:48 AM
Unknown Object (File)
Sat, Dec 7, 7:42 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

Branch
pconduit
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/conduit/call/ConduitCall.php:94XHP16TODO Comment
Unit
Tests Passed

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.