Skip to content

how to add additional query parameters in authorize URL #57

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

Closed
ssteffl opened this issue Sep 8, 2014 · 4 comments
Closed

how to add additional query parameters in authorize URL #57

ssteffl opened this issue Sep 8, 2014 · 4 comments

Comments

@ssteffl
Copy link

ssteffl commented Sep 8, 2014

How are we supposed to add query parameters. It looks like the only possible query parameter is RelayState, but I also need another query parameter (IDP requires it).

@ploer
Copy link
Contributor

ploer commented Sep 8, 2014

Interesting, I haven't run into that yet, can you share what IDP it is and what sorts of parameters it is looking for?

If you want to put together a pull request for this functionality, I'm thinking it should probably look something like a new configparameter (maybe additionalAuthorizeParams), and then getAuthorizeUrl could merge that object with the RelayState params to pass along to requestToUrl...

@ssteffl
Copy link
Author

ssteffl commented Sep 8, 2014

Site Minder

@ssteffl ssteffl closed this as completed Sep 8, 2014
@ssteffl ssteffl reopened this Sep 8, 2014
@ploer
Copy link
Contributor

ploer commented Oct 23, 2014

I can add this at some point since it sounds useful, but might not get to it for a while.

@tkopczuk
Copy link
Contributor

tkopczuk commented Nov 3, 2014

@ploer I just posted a pull request (#63) that should solve this. Tried to go along with the line of thought from your comment.

My needs actually went a bit further than that – I needed to pass different query params to 'authorize' and 'logout' endpoints, so I created 3 new options:

  • additionalParams - dict of params that are added to all requests
  • additionalAuthorizeParams - dict of params that are added to all 'authorize' requests
  • additionalLogoutParams - dict of params that are added to all 'logout' requests

Please do tell if you have any feedback, open to suggestions!

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

No branches or pull requests

3 participants