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

Add server-side user authorization #42

Open
charlesjhill opened this issue Oct 8, 2019 · 0 comments
Open

Add server-side user authorization #42

charlesjhill opened this issue Oct 8, 2019 · 0 comments
Labels
enhancement New feature or request python Pull requests that update Python code

Comments

@charlesjhill
Copy link
Owner

At this point in time, our API endpoints remain open to whomever is interested and all verification happens on the client-side. This is vulnerable to really simple exploits that would allow more tech-savvy members of the band to do whatever they want. This should be rectified by adding server-side authorization of users to all our views.

To accomplish this, we can add permission classes to our API Views; the rough idea though is to create some custom permissions that check if a user has the is_student flag set to false or has the is_staff flag set to true, and if so allows access.

@charlesjhill charlesjhill added enhancement New feature or request python Pull requests that update Python code labels Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request python Pull requests that update Python code
Projects
None yet
Development

No branches or pull requests

1 participant