Skip to content

Token scope permission class #721

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

Merged
merged 2 commits into from
Mar 12, 2013
Merged

Conversation

dulacp
Copy link
Contributor

@dulacp dulacp commented Mar 10, 2013

Like suggested, here is a token scope permission class with tests.

@dulacp dulacp mentioned this pull request Mar 10, 2013
@tomchristie
Copy link
Member

Wow. That was FAST! Nice work. :))) I'll get some docs together for this and we're done.

On 10 Mar 2013, at 15:16, Pierre Dulac [email protected] wrote:

Like suggested, here is a token scope permission class with tests.

You can merge this Pull Request by running

git pull https://github.com/dulaccc/django-rest-framework token-scope-permission
Or view, comment on, or merge it at:

#721

Commit Summary

Add TokenHasReadWriteScope class for permissions based on scopes
Add the implementation for TokenHasReadWriteScope permissions w/ oauth 1
File Changes

M rest_framework/compat.py (2)
M rest_framework/permissions.py (32)
M rest_framework/tests/authentication.py (65)
Patch Links:

https://github.com/tomchristie/django-rest-framework/pull/721.patch
https://github.com/tomchristie/django-rest-framework/pull/721.diff

@dulacp
Copy link
Contributor Author

dulacp commented Mar 11, 2013

:)

tomchristie added a commit that referenced this pull request Mar 12, 2013
@tomchristie tomchristie merged commit 12ac357 into encode:oauth Mar 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants