[ 
https://issues.apache.org/jira/browse/CXF-7935?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Colm O hEigeartaigh resolved CXF-7935.
--------------------------------------
    Resolution: Fixed

> 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
>            Assignee: Colm O hEigeartaigh
>            Priority: Major
>             Fix For: 3.3.0, 3.2.8
>
>
> 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)

Reply via email to