[jira] [Created] (CXF-7935) SAML SubjectConfirmation validation in PolicyBasedWSS4JInInterceptor

["Tomas Vanhala (JIRA)" <ji...@apache.org>]

Tomas Vanhala created CXF-7935:
----------------------------------

             Summary: SAML SubjectConfirmation validation in PolicyBasedWSS4JInInterceptor
                 Key: CXF-7935
                 URL: https://issues.apache.org/jira/browse/CXF-7935
             Project: CXF
          Issue Type: Bug
          Components: Configuration, WS-* Components
    Affects Versions: 3.2.7
            Reporter: Tomas Vanhala


WSS4J enforces the SubjectConfirmation requirements of an inbound SAML Token. For sender-vouches, a signature must be present that covers both the SOAP Body and the SAML Assertion. As explained here [http://coheigea.blogspot.com/2014/01/apache-wss4j-200-part-ii.html,] the configuration tag defined as ConfigurationConstants.VALIDATE_SAML_SUBJECT_CONFIRMATION ("validateSamlSubjectConfirmation") allows the user to switch off this validation if required.

We need to switch off the validation, in order to process non-TLS requests where only the SAML assertion is signed.

Unfortunately, it turns out that SubjectConfirmation validation cannot be switched off when using PolicyBasedWSS4JInInterceptor. It uses SamlTokenPolicyValidator, containing the following code:

 
{code:java}
if (!DOMSAMLUtil.checkSenderVouches(assertionWrapper, tlsCerts, parameters.getSoapBody(), parameters.getSignedResults())) {
    ai.setNotAsserted("Assertion fails sender-vouches requirements");
    continue;
}
{code}
This is different from WSS4JInInterceptor, which through WSSecurityEngine uses the following code:
{code:java}
if (requestData.isValidateSamlSubjectConfirmation()) {
    Element bodyElement = callbackLookupToUse.getSOAPBody();
    DOMSAMLUtil.validateSAMLResults(handlerResult, requestData.getTlsCerts(), bodyElement);
}
{code}
Please add similar handling of the VALIDATE_SAML_SUBJECT_CONFIRMATION configuration tag into PolicyBasedWSS4JInInterceptor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)