-
Notifications
You must be signed in to change notification settings - Fork 398
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
add AssertionConsumerServiceIndex to Saml2Settings.java #386
Comments
There's an ongoing effort to add support for multiple Assertion Consumer Services (see #368) which would also satisfy your request. However it has been waiting for a review by a maintainer for months, unfortunately :-( |
We are currently looking into this. @mauromol is there a good way to contact you? |
@eriktalvi I just sent you an e-mail. |
hi! any progress on this part? is there any way I can help? |
Has there been any progress on this issue? We have the same requirement. |
Hi!
According to
saml-schema-protocol-2.0.xsd
it should be possible to add an optional param called: AssertionConsumerServiceIndex(unsigned short)
For our client we need this optional param. Of course, we can fork the repository, or make a pull request on this repo, but I think you guys can do this a lot quicker.
Is it possible to add this to a v2.9.1 release?
If you want to I can also clone this repo, add a branch and create a pull-request.
Regards,
Gerard Mulder
The text was updated successfully, but these errors were encountered: