-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CAMEL-18809: RouteDefinitionHelper should resolve the intercepted fro… #8888
Conversation
🌟 Thank you for your contribution to the Apache Camel project! 🌟 If necessary Apache Camel Committers may access logs and test results in the job summaries! |
🚫 There are (likely) no components to be tested in this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I've left some comments, but they are advisory ones.
...e/camel/builder/RouteTemplateInterceptFromAndInterceptSendToEndpointWithPlaceholderTest.java
Outdated
Show resolved
Hide resolved
...e/camel/builder/RouteTemplateInterceptFromAndInterceptSendToEndpointWithPlaceholderTest.java
Show resolved
Hide resolved
...e/camel/builder/RouteTemplateInterceptFromAndInterceptSendToEndpointWithPlaceholderTest.java
Show resolved
Hide resolved
🚫 There are (likely) no components to be tested in this PR |
…m URI which is configured with property placeholder
Components tested:
|
Hello @bvahdat, please check our email conversation from yesterday: https://lists.apache.org/thread/70j2ftwdym70q5yvq6jllo07f3w693f7 I have reported another problem with id placeholders. The problem is described in my emails, and solution for fixing is proposed as well. I expect the fix would be just changing that one line. Thanks! |
@nikolag-ikor can you create a JIRA ticket |
I will look into backporting this PR into the 3.18.x branch |
…m URI which is configured with property placeholder (#8888)
…m URI which is configured with property placeholder