Skip to content

[skip release notes] Allow SP to send skip_encryption for non-encrypted SAML response (LG-3859)#4503

Merged
solipet merged 4 commits intomasterfrom
dprice-lg-3859-skip-encryption
Dec 11, 2020
Merged

[skip release notes] Allow SP to send skip_encryption for non-encrypted SAML response (LG-3859)#4503
solipet merged 4 commits intomasterfrom
dprice-lg-3859-skip-encryption

Conversation

@solipet
Copy link
Contributor

@solipet solipet commented Dec 10, 2020

This allows a configured list of SAML SP's that are configured to encrypt responses to optionally skip the encryption by passing the param skip_encryption=true with the request. This is being done so that the SAML Sinatra app can have this option, making it easier to debug SAML issues.

solipet and others added 2 commits December 10, 2020 12:17
Co-authored-by: Espartaco Palma <github@esparta.co>
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
@solipet solipet merged commit 9d42187 into master Dec 11, 2020
@solipet solipet deleted the dprice-lg-3859-skip-encryption branch December 11, 2020 03:42
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