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

Python library doesn't provide a way of handling additional parameters. #46

Closed
GoogleCodeExporter opened this issue Mar 31, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

The OAuth specification allows the service provider to accept or require
additional parameters when issuing a request token.  The service provider
would generally want to store that information in its token store for
future reference (e.g. to make decisions about resource access).

The OAuthServer / OAuthDataStore split doesn't seem to provide a way to
store this information since the OAuthDataStore.fetch_request_token()
method doesn't get passed this information.

The spec also allows for additional parameters passed to the authorisation
URL and access token URL, so it might make sense to pass those parameters
to fetch_access_token() too.  I am not sure whether
authorize_request_token() needs them.

Original issue reported on code.google.com by james.he...@gmail.com on 9 Oct 2008 at 6:18

@GoogleCodeExporter
Copy link
Author

Original comment by jmkrist...@gmail.com on 22 Oct 2008 at 5:23

  • Added labels: Lib-Python
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

For additional params along with the request token, I would recommend overriding
those methods in your own OAuthServer and OAuthDataStore.

I could update the library to accept additional params, but you'd have to 
override
those methods to use those params anyways.

Original comment by leah.culver on 14 Jan 2009 at 10:44

  • Changed state: WontFix
  • Added labels: ****
  • Removed labels: ****

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

1 participant