Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new linter: Extend MissingPermissionsChecker to examine API functions #1089

Closed
atodorov opened this issue Aug 31, 2019 · 0 comments
Closed

Comments

@atodorov
Copy link
Member

As of now this checker only examines function based or class based views skipping every other module which isn't named ".views".

API methods also require permissions but they don't look like views (b/c they aren't), e.g. they don't have a first parameter named request.

For the sake of this linter we can safely assume that API methods are in modules named api.py. They also import from modern_rpc, e.g. from modernrpc.core import rpc_method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants