servicemix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christian Connert (JIRA)" <j...@apache.org>
Subject [jira] Commented: (SMXCOMP-660) CXF-BC Component isn't JBI conform
Date Tue, 02 Feb 2010 09:22:34 GMT

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

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

Hi Freeman, 

As I have seen no progress on this and my other bug report  (SMXCOMP-670) I wanted to state
that the patch might not be ''valid'' anymore, since the SVN trunk changed. 
Further I'm a bit surprised that there seems to be no progress, as the bugs are, at least
from my point of view, realy problematic.

If you have any questions about the patch don't hesitated to contact me. My main aim was to
reuse the consumer interceptors at the provider side.

Regards

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.


Mime
View raw message