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

Introduce a New Config to Enable Logout Response Signing for IDP Init SAML SSO #4803

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

ImalshaG
Copy link
Contributor

@ImalshaG ImalshaG commented Jul 19, 2023

This PR introduces a new config to disable signing the logout response after IDP initiated SSO logins, to preserve backward compatibility.
By default, the SAML logout response for SP initiated SSO and IDP initiated SSO login flows will be signed if the Enable Response Signing property is enabled in the Service Provider. But if a customer requires the previous behaviour where the SAML logout response during IDP initiated SSO flow is not signed, the following config can be added to the deployment.toml.

[saml]
enable_saml_idp_init_logout_response_signing = false

Related Issue: wso2/product-is#16207
Related PR: wso2-extensions/identity-inbound-auth-saml#399

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5665397642

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5665397642
Status: failure

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5667255170

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5667255170
Status: success

Copy link

@jenkins-is-staging jenkins-is-staging left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/5667255170

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

Successfully merging this pull request may close these issues.

3 participants