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

Support multiple SingleLogoutService bindings #11286

Closed
emopti-jrufer opened this issue May 26, 2022 · 1 comment · Fixed by #11287
Closed

Support multiple SingleLogoutService bindings #11286

emopti-jrufer opened this issue May 26, 2022 · 1 comment · Fixed by #11287
Assignees
Labels
in: saml2 An issue in SAML2 modules type: enhancement A general enhancement
Milestone

Comments

@emopti-jrufer
Copy link
Contributor

Expected Behavior

Should be specify mulitple SingleLogoutService bindings. This would be incorporated into Saml2LogoutRequestFilter, Saml2LogoutResponseFilter and OpenSamlMetadataResolver.

Current Behavior

RelyingPartyRegistration only support adding a single singleLogoutServiceBinding.

Context

Trying to migrate off of Spring SAML Extension. Existing behavior in the Spring SAML Extension allows support for multiple SLO bindings.

Have not identified a workaround. This is preventing us from migrating since we currently support both REDIRECT and POST bindings for the SingleLogoutService and are unable to migrate all customers to utilize a single binding.

@emopti-jrufer emopti-jrufer added status: waiting-for-triage An issue we've not yet triaged type: enhancement A general enhancement labels May 26, 2022
@sjohnr sjohnr added the in: saml2 An issue in SAML2 modules label May 26, 2022
@jzheaux
Copy link
Contributor

jzheaux commented May 26, 2022

Hi, @emopti-jrufer, thanks for the suggestion and the PR. We'll take a look and see if it's feasible to add into the next milestone release.

@jzheaux jzheaux added this to the 5.8.0-M1 milestone May 26, 2022
@jzheaux jzheaux removed the status: waiting-for-triage An issue we've not yet triaged label May 26, 2022
emopti-jrufer added a commit to emopti-jrufer/spring-security that referenced this issue Jun 7, 2022
jzheaux pushed a commit that referenced this issue Jun 9, 2022
jzheaux pushed a commit that referenced this issue Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: saml2 An issue in SAML2 modules type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants