Skip to content

CAMEL-19859: camel-spring-rabbitmq set headerfilter not effective#11397

Closed
TUCJVXCB wants to merge 4 commits into
apache:camel-4.0.xfrom
TUCJVXCB:camel-4.0.x
Closed

CAMEL-19859: camel-spring-rabbitmq set headerfilter not effective#11397
TUCJVXCB wants to merge 4 commits into
apache:camel-4.0.xfrom
TUCJVXCB:camel-4.0.x

Conversation

@TUCJVXCB
Copy link
Copy Markdown
Contributor

Description

if i use java code to config spring-rabbitmq component headerfilterstrategy, the doinit() method of SpringRabbitMQComponent will be executed before i config headerfilterstrategy, so the DefaultMessagePropertiesConverter may reference headerfilterStrategy the old value。

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🐫 Maintainers, please note that first-time contributors require manual approval for the GitHub Actions to run.

⚠️ Please note that the changes on this PR may be tested automatically if they change components.

🤖 Use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

Copy link
Copy Markdown
Contributor

@orpiske orpiske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! I do have a request, though: can you please adjust the commit message so it's in the format CAMEL-19859: <text>.

It makes it easier for us when browsing the commit history 😇

@TUCJVXCB TUCJVXCB requested a review from orpiske September 15, 2023 01:58
@github-actions github-actions Bot added the core label Sep 15, 2023
@TUCJVXCB TUCJVXCB closed this Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants