Skip to content

CAMEL-14229: STOMP headers are missing in the Exchange - #3369

Merged
davsclaus merged 2 commits into
apache:masterfrom
onlybytes:CAMEL-14229
Dec 2, 2019
Merged

CAMEL-14229: STOMP headers are missing in the Exchange#3369
davsclaus merged 2 commits into
apache:masterfrom
onlybytes:CAMEL-14229

Conversation

@onlybytes

Copy link
Copy Markdown
Contributor

The response from the Exchange contains only Body. The STOMP headers are missing in the response.

@davsclaus

Copy link
Copy Markdown
Contributor

Thanks, but can you take a look and see if you can make the consumer use the header filter strategy as it should do that, eg the producer does that. But just make the filtering the reverse.

@onlybytes

onlybytes commented Nov 30, 2019

Copy link
Copy Markdown
Contributor Author

@davsclaus Thanks for taking time to review this PR. I need a clarification.
When you say make the filtering reverse, do mean selecting the accepted headers as mentioned here
OR
do you mean applyFilterToExternalHeaders?

@davsclaus

Copy link
Copy Markdown
Contributor

Oh sorry I was really just referring to making the consumer also use the header filter strategy to apply filtering, see maybe one of the other components how it does that.

@onlybytes

Copy link
Copy Markdown
Contributor Author

@davsclaus Comments incorporated. I have made consumer use the header filter strategy.

@davsclaus

Copy link
Copy Markdown
Contributor

Thanks would you be able to build a unit test also?

@onlybytes

Copy link
Copy Markdown
Contributor Author

Sure. Will give it a try.
I am working on setting custom headers. When i raise a PR for custom headers, i will include this as well

@davsclaus

Copy link
Copy Markdown
Contributor

Okay that sounds good, so will merge this, and you do another PR that incl tests, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants