davsclaus commented on PR #26942: URL: https://github.com/apache/camel/pull/26942#issuecomment-5866364897
Thanks for the detailed analysis. I've moved this to draft and asked @gnodet (who rewrote `CaseInsensitiveMap` in CAMEL-23691) to take a look. The reason: `CaseInsensitiveMap` was never meant to honour the full JDK `Map` contract. It is an internal implementation in camel-core for fast case-insensitive message headers, which are meant to be manipulated through the Camel API and the EIPs (`setHeader`, `removeHeader`, `removeHeaders`, ...), and those remove headers case-insensitively today. Removing headers through `getHeaders().keySet().remove/removeAll/retainAll` is not a use we aim to support, so the gap in the key set view is not a regression we want to fix by adding more JDK collection-view code to the map. _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]
