-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
Support for the GET /oauth/authorize is broken - returns 500 #90
Comments
maybe a hint is recent work done? Does this maybe count on a different version of authlib? |
Another clue is that the requirements.txt file has Authlib==0.14.3. That appears on the old side (from May 2020). |
OK - in current code there is: |
Well, after updating to the latest version of the library, it works a little better, but now just prints out exception NOTE - I created a CLIENT with |
Hey, I am having the same problem, did you find any solution to it? |
still fiddling but yes (sort of). Change the requirements.txt file to point
to the latest version of authlib.
…On Wed, Oct 19, 2022 at 8:52 AM anastasijamalyk ***@***.***> wrote:
Hey, I am having the same problem, did you find any solution to it?
—
Reply to this email directly, view it on GitHub
<#90 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHFVTF6YZUDI4CKL7HXEBTWD7VKLANCNFSM6AAAAAARHL7QYE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I did it, the get_consent_grant error disappeared, but not supported response type is till there |
Solved it. Here is how:
Now everything works. Re-tested flow examples in README.md and they pass. Cheers. |
This PR is to update `requirements.txt` to use the latest `Authlib`. * The `example-oauth2-server` is not fully functional until this fix goes in. For details and validation/resolution please see: authlib#90 (comment) * In addition to fixing the regression, this PR uses the latest `Authlib` (instead of pinning to a version) to encourage keeping the example working across `Authlib` enhancements -- a useful attribute of an example.
This PR is to update `requirements.txt` to use the latest `Authlib`. * The `example-oauth2-server` is not fully functional until this fix goes in. For details and validation/resolution please see: authlib#90 (comment) * In addition to fixing the regression, this PR uses the latest `Authlib` (instead of pinning to a version) to encourage keeping the example working across `Authlib` enhancements -- a useful attribute of an example.
Just using this example to explore the library, and hard to tell how things are supposed to fit together without docs or a (working) sample.
To reproduce the problem, just setup and run (flask run)
Verify
works fine/as expected.
Then try (from a web browser probably but thats what I was trying to figure out)
GET http://127.0.0.1:5000/oauth/authorize
Crashes cuz
The text was updated successfully, but these errors were encountered: