[ 
https://issues.apache.org/activemq/browse/SMXCOMP-660?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=55023#action_55023
 ] 

Christian Connert commented on SMXCOMP-660:
-------------------------------------------

Hi,

I fixed my classpath problems and run the test again, this time one by one in 
eclipse. I had the following 9 errors and no hangs:

1.) CxfBcComponentTest.testEndpointDOCWithExternalConsumerAndCxfSe -> 
assertEquals(header1.value.getId(), "retffang"); -> NullPointerException
2.) CxfBcDecoupledProviderTest.testDecoupledProvider -> CXFSe endpoint 
mtomEnabled
3-4.) CxfBcProviderMtomTest all methods ->  CXFSe endpoint mtomEnabled
5-8.) CxfBcProviderTest all methods -> CXFSe endpoint mtomEnabled
9.) CxfBCSESystemTest.testCalculatrorWithDifferentServiceEndpointNmae -> Could 
not find route for exchange (JBI route)

I need to state, that I slightly modified the maven dependencies (because I use 
Servicemix 3.3.1 and want a stable version of CXF):

Parent version 3
servicemix-shared-version 2009.01
servicemix-cxf-se-version 2009.01
cxf-version 2.2.3

This change included specifying a version for jencks (2.1), slf4j-api (1.5.8), 
slf4j-log4j12 (1.5.8), log4j (1.2.14) and removing jcl104-over-slf4j from the 
dependencies.

Thus the errors 2 - 8 are easy to explain and are caused by the 2009.01 CXF-SE 
component.
For 1 and 9 I've no explanation yet.

If I have some time I'll redo the test with the original pom.

Christian


> CXF-BC Component isn't JBI conform
> ----------------------------------
>
>                 Key: SMXCOMP-660
>                 URL: https://issues.apache.org/activemq/browse/SMXCOMP-660
>             Project: ServiceMix Components
>          Issue Type: Bug
>          Components: servicemix-cxf-bc
>    Affects Versions: servicemix-cxf-bc-2009.01
>         Environment: System:
> 2.6.28-15-generic #52-Ubuntu SMP Wed Sep 9 10:48:52 UTC 2009 x86_64 GNU/Linux
> JAVA:
> java version "1.6.0_16"
> Java(TM) SE Runtime Environment (build 1.6.0_16-b01)
> Java HotSpot(TM) 64-Bit Server VM (build 14.2-b01, mixed mode)
> Server:
> apache-tomcat-6.0.14 
> ServiceMix:
> 3.3.1 war
>            Reporter: Christian Connert
>            Assignee: Freeman Fang
>            Priority: Critical
>         Attachments: cxfbc.patch
>
>
> Hi,
> I just noticed that the CXF-BC Component ain't standard conform. There are 
> several problems:
> 1.) The JbiInInterceptor is supposed to write the (soap) headers to the 
> NormalizedMessage (javax.jbi.messaging.protocol.headers). Accordingly to the 
> JBI Spec. the headers should be stored within a 
> Map<String,org.w3c.dom.DocumentFragment>. But in fact the JbiInInterceptor 
> just puts the header.getObject() in the map. Which results in a 
> ClassCastException in other components (like JMS). The objects are some DOM 
> ElementImpl.
> 2.) I'm very unhappy with the fix 
> (https://issues.apache.org/activemq/browse/SMXCOMP-203 states) which removes 
> the WS-Security header from the propagated headers. It's not necessary that 
> the security is implemented with cxf. Thus I think that this fix is not 
> correct. It would be better to e.g. use some other saaj implementation (for 
> instance axis2). Further it might help to at some xerces jar into the class 
> path, but I haven't tested it yet.
> 3.) If useJBIWrapper==true the JbiInWsdl1Interceptor warps the message within 
> JBI parts. That is correct. If headers are specified in the WSDL then these 
> headers are added as separate parts. But it never removes the headers, which 
> where added as part, from the SoapMessage, thus the JbiInInterceptor will put 
> them into the javax.jbi.messaging.protocol.headers map. I think this is not 
> like it should be. Servicemix soap does remove the headers form the 
> underlying message and only puts unspecified (not part of the WSDL) headers 
> into the javax.jbi.messaging.protocol.headers map.
> kind regards
> Christian

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to