On Friday 17 December 2010 4:32:12 am Jim Ma wrote: > 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 ?
Yea. Exactly that. I'm not sure why my test run didn't fail though. Bizzarre. Must of somehow picked up the older version or something. Hmm.... Thanks! Dan > 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 -- Daniel Kulp dk...@apache.org http://dankulp.com/blog
