[
https://issues.apache.org/jira/browse/CAMEL-12069?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16280810#comment-16280810
]
ASF GitHub Bot commented on CAMEL-12069:
----------------------------------------
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/camel/pull/2139
CAMEL-12069: ActiveMQ/JMS component: transferExchange option does not…
… transfer exchange properties
TransferExchange now includes exchange properties again. This restores the
previous behavior accidentally broken by
5dd59162e4ac9335e497934b8d662f598cb779d1.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/PascalSchumacher/camel
CAMEL-12069-transferExchange-should-include-exchange-properties
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/2139.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2139
----
commit 86b94fc4d937eff3d7e1031fc95477c3042579b2
Author: Pascal Schumacher <[email protected]>
Date: 2017-12-06T19:58:02Z
CAMEL-12069: ActiveMQ/JMS component: transferExchange option does not
transfer exchange properties
TransferExchange now includes exchange properties again. This restores the
previous behavior accidentally broken by
5dd59162e4ac9335e497934b8d662f598cb779d1.
----
> ActiveMQ/JMS component: transferExchange option does not transfer exchange
> 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.
> The following fields are transferred: In body, Out body, Fault body, In
> headers, 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.isAllowSerializedHeaders())_ 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)
