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

OAuth 1.0a authentication #678

Closed
wants to merge 12 commits into from

Conversation

swistakm
Copy link
Contributor

OAuthAuthentication class as discussed in #8.

It usues django-oauth-plus and oauth2 packages. This backend is optional because strongly depends on external libs. This is a reason I don't add new packages to requirements.txt.

Added new tests, Auth class and minimal documentation. I changed travis configs too.

In next days i will provide more detailed documentation but at first I will try to add more structure for existing django-oauth-plus docs.

@tomchristie
Copy link
Member

Wonderful, wonderful, wonderful. Will review this properly later but looks great. Excellent stuff!

@dbrgn
Copy link
Contributor

dbrgn commented Feb 25, 2013

Yay for OAuth support :) Looking forward to this.

@@ -9,6 +9,18 @@
from rest_framework.authtoken.models import Token
import base64

from django.core.exceptions import ImproperlyConfigured
try:
import oauth2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use import oauth2 as oauth

…r issues

- alias oauth2 as oauth
- remove rouge print
- remove docstring markups
- OAuthAuthentication.authenticate() now returns (user, token) two-tuple on success
- don't set request.user because it's already set
oauth_provider can be added to INSTALLED_APPS only if these packages are installed
@@ -14,6 +14,8 @@ env:
install:
- pip install $DJANGO
- pip install defusedxml==0.3
- pip install oauth2
- pip install django-oauth-plus
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs to only install these on Python 2.x

@tomchristie
Copy link
Member

Superseded by #709 (Which includes this work)

Make any further pull requests against the oauth branch, and we can merge that into master when we're happy it's all good to go.

Thanks again for this - looking really good!

@tomchristie tomchristie closed this Mar 7, 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.

3 participants