Skip to content

Add the possibility to continue if no encryption cert is available#2208

Merged
tvdijen merged 2 commits intomasterfrom
bugfix/encrypted-assertions
Aug 23, 2024
Merged

Add the possibility to continue if no encryption cert is available#2208
tvdijen merged 2 commits intomasterfrom
bugfix/encrypted-assertions

Conversation

@tvdijen
Copy link
Copy Markdown
Member

@tvdijen tvdijen commented Aug 21, 2024

Continuation of #2204

@tvdijen tvdijen requested a review from thijskh August 21, 2024 19:57
@monkeyiq
Copy link
Copy Markdown
Contributor

Would it make sense to mention encryption.optional from the assertion.encryption documentation in a Note that encryption may optionally, not done by default, be lifted for an SP that does not supply a certificate?

I imagine the person who is setting up a config will notice all of these things but somebody who inherits the maintenance of an SSP install and is reading the docs might be in a better position to know about encryption.optional if it is mentioned from the main assertion.encryption documentation.

@tvdijen tvdijen force-pushed the bugfix/encrypted-assertions branch from 366805b to 9673abb Compare August 22, 2024 06:37
@tvdijen tvdijen requested a review from monkeyiq August 22, 2024 06:43
@tvdijen tvdijen merged commit f99217b into master Aug 23, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 22, 2024
@tvdijen tvdijen deleted the bugfix/encrypted-assertions branch March 10, 2026 19:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants