cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Glynn, Eoghan" <eoghan.gl...@iona.com>
Subject RE: svn commit: r439963 - in /incubator/cxf/trunk: common/src/main/java/org/apache/cxf/staxutils/ rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/ rt/bindings/soap/src/test/java/org/apache/cxf/binding/soap/ rt/core/src/main/java/org
Date Fri, 08 Sep 2006 07:45:34 GMT

Hi Freeman,

I was a bit confused by your change to the SoapOutInterceptor below,
which seems to only allow the header parts to be mbe added if no
previous interceptor has added a header.

Obviously this would not be the case when the WS-A interceptor adds
wsa:* headers. 

Can you review the change I committed to SoapOutInterceptor to allow
interceptor-provided headers and message-specific headers to both be
marshalled.

Cheers,
Eoghan

> Modified: 
> incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/
> cxf/binding/soap/interceptor/SoapOutInterceptor.java

[snip]

>              Element eleHeaders = message.getHeaders(Element.class);
> -            
> -
>              if (eleHeaders != null) {
>                  StaxUtils.writeElement(eleHeaders, xtw, true);
> -            }
> +            } else {
>              
> +                handleHeaderPart(message);
> +            }


Mime
View raw message