Mailing-List: contact issues-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Date: Wed, 6 Dec 2017 20:02:00 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@camel.apache.org
Message-ID: <JIRA.13123226.1512576452000.402103.1512590520150@Atlassian.JIRA>
In-Reply-To: <JIRA.13123226.1512576452000@Atlassian.JIRA>
References: <JIRA.13123226.1512576452000@Atlassian.JIRA> <JIRA.13123226.1512576452629@jira-lw-us.apache.org>
Subject: [jira] [Commented] (CAMEL-12069) ActiveMQ/JMS component:
 transferExchange option does not transfer exchange properties
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
archived-at: Wed, 06 Dec 2017 20:02:05 -0000


    [ https://issues.apache.org/jira/browse/CAMEL-12069?page=3Dcom.atlassia=
n.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D162=
80809#comment-16280809 ]=20

ASF GitHub Bot commented on CAMEL-12069:
----------------------------------------

PascalSchumacher opened a new pull request #2139: CAMEL-12069: ActiveMQ/JMS=
 component: transferExchange option does not=E2=80=A6
URL: https://github.com/apache/camel/pull/2139
=20
=20
   =E2=80=A6 transfer exchange properties
  =20
   TransferExchange now includes exchange properties again. This restores t=
he previous behavior accidentally broken by 5dd59162e4ac9335e497934b8d662f5=
98cb779d1.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
=20
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> ActiveMQ/JMS component: transferExchange option does not transfer exchang=
e properties
> -------------------------------------------------------------------------=
------------
>
>                 Key: CAMEL-12069
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12069
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-activemq, camel-jms
>    Affects Versions: 2.20.1
>            Reporter: Pascal Schumacher
>
> The ActiveMQ/JMS component documenation generated from the _JMSComponent#=
setTransferExchange_ javadoc states:
> {quote}
> You can transfer the exchange over the wire instead of just the body and =
headers.=20
> The following fields are transferred: In body, Out body, Fault body, In h=
eaders, Out headers, Fault headers, exchange properties, exchange exception=
.
> {quote}
> But exchange properties are *not* transfered.
> I debuged into this and noticed that _JMSBinding#createJmsMessage_ calls =
_DefaultExchangeHolder.marshal(exchange, false, endpoint.isAllowSerializedH=
eaders())_ when _transferExchange_ is _true_. The _false_ parameter is the =
reason that the exchange properties are not included in the JMS message.
> This should either be fixed or the documentation should be updated.


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)