-
-
Notifications
You must be signed in to change notification settings - Fork 424
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
xAuth support? #191
Comments
Interesting. Right now, I absolutely don't: however in a week or so I'll have much more resource freed up to pursue this. If you're interested in trying the work yourself, the first thing to do is to understand how xAuth differs from OAuth, and whether oauthlib contains all the stuff we need to make this work properly. If it does, then you can make the changes directly here: if not, you'll need to make changes there first. |
Thanks for the response @Lukasa. I've poked around in the docs and source code for both libraries, but with limited to time to devote to this, it's not obvious to me how to make this work or where any changes would need to be made, so I'm going to have to use the other library for now. Essentially, in xAuth three parameters need to be added to an initial oAuth 2.0 request to get the access token to make requests:
Here's what the HTTP request headers for successful session look like (using the library I mentioned above) to get a list of articles in an Instapaper account: Get access token:
Get list of articles:
If you're interested in following up on this, let me know how else I can help. |
Some Twitter applications (see https://dev.twitter.com/oauth/xauth) use the "xAuth" variant of oAuth, as does the full Instapaper API: https://www.instapaper.com/api/full
This implementation of the Instapaper API uses python-oauth2 and actually works properly: https://github.com/rsgalloway/instapaper/blob/master/instapaper.py#L132
(See also https://github.com/rsgalloway/instapaper#basic-usage)
But it's not clear to me how to adapt requests-oauthlib to accomplish this. Any pointers?
The text was updated successfully, but these errors were encountered: