-
Notifications
You must be signed in to change notification settings - Fork 12
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 custom params #3
Comments
I would be open to this. Feel free to make a PR. If you can't do that, could you please provide an example of how you think this might look? For example - are you looking for an optional argument that can be passed to the |
Could you please also provide a link to the documentation and any specifications for these authorization adapters? It might be worth while to have a different package for adapters if this is something others will need to use with different services. |
Here is what other clients are doing in this regard - just letting user configure whatever they want. For PingFederate sometimes you need to add the following query param to the authorize endpoint call.
This is just an example of course, and there are different adapters that can be used. |
You can see it in the docs here - https://docs.pingidentity.com/bundle/pingfederate-93/page/nfr1564003024683.html. |
Thanks. I probably won't get to this over the holiday, but I will work on it soon. |
Would be good to be able to send custom query params on the calls to the auth server. Some servers like PingFederate support the concept of authorization adapters, which should be set by the OAuth client.
The text was updated successfully, but these errors were encountered: