Missing SubjectConfirmationData must not break validation#825
Missing SubjectConfirmationData must not break validation#825m0ark wants to merge 1 commit intosimplesamlphp:masterfrom
Conversation
SubjectConfirmationData is optional , but must not break validation if absent!
|
Jumping in here...SubjectConfirmationData is NOT optional for SSO, at least according to the SAML2 Profiles specification. I must be missing something. |
|
Well, according to the SAML2 SSO Profiles you are right, it is not optional. |
|
FWIW, I believe the Profiles specification takes precedence here. |
|
Actually I tried to get module-attributeaggregator working with encrypted assertions by using this function to decrypt and validate the response received after querying the attribute authority. |
SubjectConfirmationData is optional , but must not break validation if absent!