-
Notifications
You must be signed in to change notification settings - Fork 94
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
Setting a state variable #47
Comments
Not without modifying the code a little bit. nginx-openid-connect/openid_connect.js Line 262 in db991ec
If PKCE is in use the state value is already be in use for the What kind of value would you like to send to the IdP in the state? |
I am currently using PKCE, apart from that I need to send some additional information. As you suggested, I think it would be better if we are able to pass parameters along with the request |
If it`s just about sending additional URI params to the IdP you can simply change the NGINX configuration and send them along.
Handling new params at
Whats the usecase? This would help better to undestand where you should implement the new logic. |
Even if I disable the PKCE. state variable is set to default value 0 so I am unable to set the custom parameter there. I want to send a custom UUID parameter to IDP and get that parameter to redirect to the proper subdomain. IDP is only allowing the state parameter it is not accepting the custom query parameters. |
@tippexs If I open a PR to support customizing the state parameter and if PKCE is not enabled then use the given state parameter will that be okay ? |
If we want to attach new URI parameters for the codexch is it possible ? |
Is there a way to pass a state variable ?
The text was updated successfully, but these errors were encountered: