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.

Sign up for GitHub

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

Jump to bottom

Add support for multiple Assertion Consumer Services (ACS) #328

Open
mauromol opened this issue Apr 9, 2021 · 0 comments · May be fixed by #368
Open

Add support for multiple Assertion Consumer Services (ACS) #328

mauromol opened this issue Apr 9, 2021 · 0 comments · May be fixed by #368

Comments

@mauromol
Copy link
Contributor

mauromol commented Apr 9, 2021

This is an enhancement request to make java-saml support multiple Assertion Consumer Services.
This means:

  • the ability to configure multiple ACSes (URL and binding, with index and the indication of which is the default one) in the settings, with corresponding metadata rendering
  • the ability to select which ACS the response should be targeted to when creating an AuthnRequest

I can provide a PR for this. Here is a preview: mauromol@54a1459
It's not ready for opening a PR though, because the work is based on previous work (mainly #307 and #315).
mauromol added a commit to mauromol/java-saml that referenced this issue Nov 9, 2021
@mauromol
Add support for multiple Assertion Consumer Services (ACS)
21e7b45 
This change is 100% backward compatible with the old way of specifying
just one ACS, both from a configuration and from an API point of view.
The generated metadata and AuthnRequest XMLs are also exactly the same
as before when just one ACS is specified with non-indexed properties.

Fixes SAML-Toolkits#328.
@mauromol mauromol linked a pull request Nov 9, 2021 that will close this issue
Open
mauromol added a commit to mauromol/java-saml that referenced this issue Nov 28, 2021
@mauromol
Add support for multiple Assertion Consumer Services (ACS)
918fddf 
This change is 100% backward compatible with the old way of specifying
just one ACS, both from a configuration and from an API point of view.
The generated metadata and AuthnRequest XMLs are also exactly the same
as before when just one ACS is specified with non-indexed properties.

Fixes SAML-Toolkits#328.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for multiple Assertion Consumer Services (ACS) mauromol/java-saml
1 participant
@mauromol