Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAGvJFueMsyCuUyaYYfQtqhjHNwhjZ4qSk=5LqoYYjDVgwJx5DQ@mail.gmail.com>
References: 
 <CAGvJFucafr2MVw6Mb+wsYi82R_VPpDS_q7Nws9pjZfV5LDZG4Q@mail.gmail.com>
	<CAGvJFueMsyCuUyaYYfQtqhjHNwhjZ4qSk=5LqoYYjDVgwJx5DQ@mail.gmail.com>
Date: Fri, 24 Jul 2015 16:27:51 +0200
Message-ID: 
 <CAF8t5XsGwm6ju4=jCgxqMd2UbcTKx-J5m9xwe+jkAqiasYsSAw@mail.gmail.com>
Subject: Re: To CXF experts: Setting of SOAP headers via the Camel Header
 "org.apache.cxf.headers.Header.list" for CXF data format "PAYLOAD"
From: Aki Yoshida <elakito@gmail.com>
To: "dev@camel.apache.org" <dev@camel.apache.org>
Content-Type: text/plain; charset=UTF-8

Hi Franz,
I wanted to look at this but had not had time and a little late to comment.
One question or concern that I had wasthat this would change the
behavior of the existing tunneling scenarios using some soap headers,
no?
In other other words, some unwanted headers might tunnel through and
show up at the other end?
thanks.
regards, aki

2015-07-21 12:44 GMT+02:00 Franz Paul Forsthofer <emc2fpf@googlemail.com>:
> Hi,
>
> can I assume that there are no objections against the patch I proposed
> in  https://issues.apache.org/jira/browse/CAMEL-8978?
>
> Best Regards Franz
>
> On Fri, Jul 17, 2015 at 10:02 AM, Franz Paul Forsthofer
> <emc2fpf@googlemail.com> wrote:
>> Hi,
>>
>> Accessing the SOAP headers via the Camel header
>> "org.apache.cxf.headers.Header.list" after a CXF consumer works fine,
>> also for the CXF data format "PAYLOAD".
>>
>> However, currently the setting of SOAP headers via the Camel header
>> "org.apache.cxf.headers.Header.list" is not working for a CXF producer
>> with  CXF data  format "PAYLOAD".
>>
>> I found out that the list value contained in Camel header
>> "org.apache.cxf.headers.Header.list"  is correctly forwarded to the
>> CXF request, however in the CxfEndpoint.CamelCxfClientImpl this list
>> is overwritten by the value CxfPaylaod.getHeaders().
>>
>> Therefore I propose  to merge the values from the Camel header
>> "org.apache.cxf.headers.Header.list" with the values of
>> CxfPaylaod.getHeaders(). This has the advantage that you can set the
>> headers either via the CxfPayload or by the Camel header
>> "org.apache.cxf.headers.Header.list".
>>
>> For more details have a look at https://issues.apache.org/jira/browse/CAMEL-8978
>>
>> Can somebody of the CXF experts comment whether my solution is
>> feasible. Then I will commit the patch.
>>
>> Best Regards
>>
>> Franz Forsthofer