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.f...@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 > >
