oscerd commented on PR #26748: URL: https://github.com/apache/camel/pull/26748#issuecomment-5811307219
Thanks for the thorough re-review @davsclaus. On the two non-blocking notes: - 🟡 **default not shown in the catalog** — agreed the object-typed option can't carry a `defaultValue`. The new default is documented in the 4.23 upgrade guide (the canonical place for a changed default). Folding the same sentence into the `@UriParam` description is a good idea, but it triggers a full catalog/DSL regen and the shared local `~/.m2` is currently mixed with newer-main components, so I'm holding it as a trivial follow-up rather than churn this otherwise-ready branch — happy to add it before merge if you'd rather it also live in the component docs. - 🔵 **null guards** — agreed, and intended: with the eager init an explicit `setHeaderFilterStrategy(null)` now sticks and disables filtering entirely, which is coherent with supplying a permissive custom strategy. Left as-is. _Claude Code on behalf of Andrea Cosentino (@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]
