davsclaus opened a new pull request, #24845: URL: https://github.com/apache/camel/pull/24845
## Summary `JsonDataFormatReifier.prepareDataFormatConfig` unconditionally put 10 Jackson-only options (`include`, `allowJmsType`, `collectionType`, `useList`, `combineUnicodeSurrogates`, `moduleClassNames`, `moduleRefs`, `enableFeatures`, `disableFeatures`, `allowUnmarshallType`) plus `objectMapper` into the property map for all JSON libraries. When any of these was non-null and the library was Gson, Fastjson, or Jsonb, `PropertyBindingSupport` with `withMandatory(true)` threw a `PropertyBindingException` at route startup. - Moved the 10 Jackson-only options and `objectMapper` into the `library == Jackson` guard block - Added a separate `library == Jsonb` guard for `objectMapper` (the only non-Jackson library that supports it) - Fixed pre-existing `@Metadata(includeProperties)` casing bug in `FastjsonDataFormat`: `prettyprint` → `prettyPrint` so the generated configurer recognizes the property - Regenerated Fastjson configurer and metadata ## Test plan - [x] Added `JsonDataFormatReifierTest` with 4 tests verifying Gson/Fastjson/Jsonb don't get Jackson-only options, and Jackson still gets all of them - [ ] CI green _Claude Code on behalf of davsclaus_ Co-Authored-By: Claude Opus 4.6 <[email protected]> -- 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]
