-
Notifications
You must be signed in to change notification settings - Fork 141
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
Use form_post instead of fragment if supported by the provider #4
Labels
enhancement
New feature or request
Comments
1 task
See #13 for reasons to keep |
@joshcanhelp the point here was to default to form_post when its supported, that is still something we should do. Please re-open. |
It already defaults to https://github.com/auth0/express-openid-connect/blob/master/lib/config.js#L6-L10 App fails on first load if that's invalid. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We could use
form_post
instead offragment
forfragment
response types IF the provider lists form_post as supported.The text was updated successfully, but these errors were encountered: