oscerd commented on PR #29627: URL: https://github.com/apache/camel/pull/29627#issuecomment-6819042418
Pushed `CAMEL-26853: Address review feedback - the headers of the delegated HTTP request are not copied to the result either` (55f8bbac5140) on top of 605f8bb9aa33: 1. `CamelHttpUri`, `CamelHttpQuery`, `CamelHttpMethod` and `Accept` are removed from the HTTP exchange before the copy-back, next to the generated `Authorization`, so a later HTTP step in the route no longer inherits the OData target and method; a caller value of these headers on the exchange is kept. `testBasicAuthentication` asserts all four are absent from the result, and the upgrade guide entry says so. 2. `testConfiguredRequestHeadersWhenHttpRequestFails` stays as the guard you describe. 3. The `Accept-Encoding: identity` header in the test now carries its reason: it keeps WireMock from gzipping the body, so the response headers are the plain ones. camel-odata module: 49 tests, 0 failures. Review re-requested. _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]
