cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Ma <mail2ji...@gmail.com>
Subject Re: Does anybody also see OOBHeaderTest failure
Date Fri, 17 Dec 2010 09:32:12 GMT
Hi Freeman ,

This failure is introduced in this commit :
http://svn.apache.org/viewvc?rev=1050167&view=rev.
Dan,  is your change in rev. 1050167 with Freeman's modification are
what you meant to optimize ?

Thanks,
Jim

On Fri, Dec 17, 2010 at 4:25 PM, Freeman Fang <freeman.fang@gmail.com> wrote:
> Ok, we also need add
> soapMessage.getInterceptorChain().add(ending);
> in checkUltimateReceiverHeaders method after a SoapFault set to SoapMessage
> and so that this fault could be thrown after OnewayProcessorInterceptor.
>
> Create CXF-3200[1] to track it and will commit fix soon.
>
> [1]https://issues.apache.org/jira/browse/CXF-3200
>
> Freeman
>
> On 2010-12-17, at 下午3:31, Freeman Fang wrote:
>
>> Hi,
>>
>> This test also failed on Hudson CI, so I investigate it a little bit and
>> found that this commit[1] cause this problem.
>>
>> Before this change, the MustUnderstandEndingInterceptor is added for
>> server side in any case, after this change, MustUnderstandEndingInterceptor
>> is only added for server side when
>> notUnderstandHeaders isn't empty. Not sure current logic is wrong or what
>> we need is just refactor the wrong testcase.
>>
>>
>>
>> [1]http://svn.apache.org/viewvc/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/MustUnderstandInterceptor.java?r1=1050165&r2=1050164&pathrev=1050165
>>
>> Freeman
>> On 2010-12-17, at 下午1:24, Freeman Fang wrote:
>>
>>> Guys,
>>>
>>> I saw this failure today from my side, any guy else also saw it?
>>>
>>> Regards
>>> Freeman
>>>
>>> --
>>> Freeman Fang
>>>
>>> ------------------------
>>>
>>> FuseSource: http://fusesource.com
>>> blog: http://freemanfang.blogspot.com
>>> twitter: http://twitter.com/freemanfang
>>> Apache Servicemix:http://servicemix.apache.org
>>> Apache Cxf: http://cxf.apache.org
>>> Apache Karaf: http://karaf.apache.org
>>> Apache Felix: http://felix.apache.org
>>>
>>
>>
>> --
>> Freeman Fang
>>
>> ------------------------
>>
>> FuseSource: http://fusesource.com
>> blog: http://freemanfang.blogspot.com
>> twitter: http://twitter.com/freemanfang
>> Apache Servicemix:http://servicemix.apache.org
>> Apache Cxf: http://cxf.apache.org
>> Apache Karaf: http://karaf.apache.org
>> Apache Felix: http://felix.apache.org
>>
>
>
> --
> Freeman Fang
>
> ------------------------
>
> FuseSource: http://fusesource.com
> blog: http://freemanfang.blogspot.com
> twitter: http://twitter.com/freemanfang
> Apache Servicemix:http://servicemix.apache.org
> Apache Cxf: http://cxf.apache.org
> Apache Karaf: http://karaf.apache.org
> Apache Felix: http://felix.apache.org
>
>

Mime
View raw message