CAMEL-10666: exposed uri param to enable inclusion of serializable headers#1370
Closed
davidecavestro wants to merge 1 commit intoapache:camel-2.17.xfrom
davidecavestro:CAMEL-10666
Closed
CAMEL-10666: exposed uri param to enable inclusion of serializable headers#1370davidecavestro wants to merge 1 commit intoapache:camel-2.17.xfrom davidecavestro:CAMEL-10666
davidecavestro wants to merge 1 commit intoapache:camel-2.17.xfrom
davidecavestro:CAMEL-10666
Conversation
…eaders should be included
davidecavestro
commented
Jan 3, 2017
| if (endpoint != null && endpoint.isTransferExchange()) { | ||
| LOG.trace("Option transferExchange=true so we use JmsMessageType: Object"); | ||
| Serializable holder = DefaultExchangeHolder.marshal(exchange); | ||
| Serializable holder = DefaultExchangeHolder.marshal(exchange, false, endpoint.isAllowSerializedHeaders()); |
Contributor
Author
There was a problem hiding this comment.
The hard coded false argument is there to avoid serializing exchange props, as it seems they may contain some instances of non serializable classes, i.e. for exchange history tracking purposes.
BTW if needed it could be easily exposed an additional parameter to control even this behavior.
Contributor
|
@davidecavestro can you please open this PR against master? We don't add new option on minor release. Thanks. |
Contributor
Author
zregvart
pushed a commit
to zregvart/camel
that referenced
this pull request
Jul 9, 2021
[ENTESB-13186] CAMEL-14792: Ensure CAMEL-13468 works without sacrific…
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As per CAMEL-10666 this is an enhancement proposal to restore the possibility of saving Serializable headers within the exchange, simply setting to true the new parameter allowSerializedHeaders.