-
-
Notifications
You must be signed in to change notification settings - Fork 423
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
OAuth2Session ignores self.state when generation an authorization url #57
Comments
I agree. A way both could be achieved would be to combine the
@FelixSchwarz fancy making those changes and sending a PR? Don't worry if you don't have time, I can find some soonish. |
Am 18.06.2013 16:57, schrieb Ib Lundgren:
Well currently I have approximately zero free time so I'd be glad if you could |
|
OAuth2Session always calls
self.new_state()
inauthorization_url
even if the user specified a state in the constructor. I think that should be changed.As a side note I think the
state
andstate_generator
arguments should be mutually exclusive to prevent errors on the user side.The text was updated successfully, but these errors were encountered: