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

Adding additional query params breaks the client id #39

Closed
jshthornton opened this issue Jul 24, 2021 · 2 comments · Fixed by #40
Closed

Adding additional query params breaks the client id #39

jshthornton opened this issue Jul 24, 2021 · 2 comments · Fixed by #40

Comments

@jshthornton
Copy link
Contributor

Google's authorization url let's you supply an additional query param of ?prompt=consent to force users to have to reauth every time.

If you supply that in the authorization url as such https://accounts.google.com/o/oauth2/v2/auth?prompt=consent then the client id is not appended and Google rejects with a malformed url error.

This can be bypassed by forcefully inserting the client id in the authorization url.

@bhubr
Copy link
Owner

bhubr commented Aug 25, 2021

@all-contributors please add @jshthornton for code

@allcontributors
Copy link
Contributor

@bhubr

I've put up a pull request to add @jshthornton! 🎉

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 a pull request may close this issue.

2 participants