davsclaus commented on PR #26931: URL: https://github.com/apache/camel/pull/26931#issuecomment-5866365250
Thanks for the detailed analysis. I've moved this to draft and asked @gnodet (who rewrote `CaseInsensitiveMap` in CAMEL-23691) to decide. The reason: `CaseInsensitiveMap` is an internal implementation in camel-core for fast case-insensitive message headers, manipulated through the Camel API and EIPs, and not meant to follow the full JDK `Map` contract, so the javadoc wording alone is not a reason to change it. What makes this one different is that the key case is visible through the Camel API: a header set as `content-type` is sent on by a component as `Content-Type`, which matters for transports with case-sensitive header names such as Kafka. The folding to the canonical constant was deliberate in CAMEL-23691 (it has its own test), so @gnodet should confirm whether it was only meant as a memory optimisation, in which case this fix is fine, or whether the canonical case is intended. _Claude Code on behalf of davsclaus_ -- 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]
