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

Multiple IdPs for Pedestal-sp #4

Open
simongray opened this issue Nov 17, 2020 · 0 comments
Open

Multiple IdPs for Pedestal-sp #4

simongray opened this issue Nov 17, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@simongray
Copy link
Member

Currently, the Pedestal-sp configuration expects a single IdP as we just need to integrate with WAYF for now - and possibly forever. Modifying the code to handle multiple IdPs would entail

  • specifying these as a list or map with identifiers in the config map
  • redirecting to the correct IdP based on an identifier supplied as a query parameter to the GET /saml endpoint
  • using the correct IdP certificate to validate the response to the POST /saml endpoint
    • Currently the RelayState is just used for the internal redirect done after validation. However, since this query param is going to be present in the IdP response, this is one conceivable way to use match the response with the correct IdP certificate during validation. The main trade-off is that it complects the value of RelayState which must then be parsed to separate the redirect URL from the IdP identifier.

Obviously, code changes supporting multiple IdPs will somewhat complect the entirety of the setup no matter what. If Pedestal-sp is to be spun off, it should probably have this functionality.

@simongray simongray added the enhancement New feature or request label Nov 17, 2020
@simongray simongray changed the title Long shot: multiple IdPs for Pedestal-sp Multiple IdPs for Pedestal-sp Nov 20, 2020
@simongray simongray transferred this issue from kuhumcst/glossematics Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant