oscerd opened a new pull request, #26034: URL: https://github.com/apache/camel/pull/26034
## Backport of #25571 to `camel-4.18.x` Cherry-pick of `ba5aced7f6a1` (CAMEL-24418) onto `camel-4.18.x`. **Original PR:** #25571 — CAMEL-24418: camel-http/http-common/netty-http/undertow/vertx-http - do not resolve property placeholders in HTTP URI override headers **Original author:** @oscerd **JIRA:** [CAMEL-24418](https://issues.apache.org/jira/browse/CAMEL-24418) ### What changed The HTTP producers no longer resolve property placeholders (`{{...}}`) on the message-supplied endpoint-URI override headers `CamelHttpUri` and `CamelRestHttpUri`. Placeholders written in the route's endpoint URI continue to resolve at build time exactly as before. ### Not a straight cherry-pick — two adjustments This branch has diverged from `main`, so two changes were made beyond the pick. Reviewers should look at these rather than assuming a mechanical port: 1. **`camel-vertx-http` has an eighth site that `main` does not have.** `VertxHttpHelper.resolveQueryString()` also resolved placeholders on the `CamelHttpUri` header value. `main` has no such call there, so the upstream patch does not touch it and a plain cherry-pick would have left it live. It is removed here, so every site on this branch is covered rather than only the ones the upstream patch happens to reach. 2. **The `camel-netty-http` import hunk conflicted.** `RuntimeExchangeException` is unused after the change and is dropped. The incoming `DeserializationFilterHelper` import is **not** taken — that class is `main`-only and does not exist on 4.18.x, so importing it would break the build. Everything else applied cleanly. Both tests carried over without modification: `HttpUriHeaderPlaceholderTest` extends nothing, and `BaseJettyTest` on this branch already carries the JUnit 5 import it needs. The upgrade-guide entry is intentionally **not** part of the backport: the guides for every release line live on `main`, so the 4.18 note is added to `camel-4x-upgrade-guide-4_18.adoc` there instead. ### Verification Full reactor `mvn clean install -DskipTests` on this branch. --- 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01QDvCTjCXTvWTZxJQws95UH -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
