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
Unknown Object (File)
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)
Dec 15 2024, 4:35 PM
Unknown Object (File)
Dec 14 2024, 8:19 AM
Unknown Object (File)
Dec 13 2024, 8:49 PM
Unknown Object (File)
Dec 11 2024, 5:48 AM
Unknown Object (File)
Dec 7 2024, 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

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.