oscerd opened a new pull request, #25833: URL: https://github.com/apache/camel/pull/25833
Fixes [CAMEL-24455](https://issues.apache.org/jira/browse/CAMEL-24455). isHttpProxy() tested path.startsWith(PROXY_PATH), so any endpoint whose path merely began with "proxy" - proxyStats, proxy-health, proxying - was treated as the documented platform-http:proxy endpoint. That is not only a naming curiosity: getPath() returns "/" for such an endpoint, making it a catch-all, and VertxPlatformHttpConsumer.handleProxy() sets Exchange.HTTP_HOST from the request's own Host header so a bridging producer forwards there. A route author naming an endpoint proxyStats got a catch-all whose forward target came from the caller. Compare for equality. The check is deliberately strict rather than tolerating a leading slash: platform-http:/proxy did not select proxy mode before and still does not, so tightening this can never turn an endpoint into a proxy that was not already one. The test asserts that, so the check is not loosened later by mistake. Every platform-http:proxy usage in the tree - the component docs, PlatformHttpProxyTest, VertxPlatformHttpProxyTest, VertxPlatformHttpsProxyTest - already uses the exact path. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> Signed-off-by: Andrea Cosentino <[email protected]> \n## Verification\n\n3 files changed, including 1 test file(s). Module build with \`-am\` is green on current main, no generated-file drift. Verified against the pre-fix code when the change was written. _Claude Code on behalf of oscerd_ -- 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]
