Croway opened a new pull request, #24812:
URL: https://github.com/apache/camel/pull/24812

   ## Summary
   
   _Claude Code on behalf of Croway_
   
   Fixes [CAMEL-24165](https://issues.apache.org/jira/browse/CAMEL-24165) — the 
`ignoreUnmarshalledHeaders` option on `SoapDataFormat` is completely unusable 
due to two bugs introduced with CAMEL-22354:
   
   - **Problem A — StackOverflowError**: The builder's `boolean` overload at 
`SoapDataFormat.Builder.ignoreUnmarshalledHeaders(boolean)` calls 
`Boolean.valueOf(boolean)` which returns a `Boolean` that auto-unboxes back to 
`boolean`, hitting the same overload recursively. Fixed by using 
`Boolean.toString(boolean)` to route to the `String` overload, matching the 
pattern used by every other builder in the package.
   
   - **Problem B — Reifier silently drops the option**: 
`SoapDataFormatReifier.prepareDataFormatConfig()` never maps 
`ignoreUnmarshalledHeaders` into the properties map, so the setting never 
reaches the runtime data format regardless of DSL. Fixed by adding the missing 
`properties.put(...)` call, following the same pattern as 
`JaxbDataFormatReifier` and `JsonDataFormatReifier`.
   
   ## Test plan
   
   - [x] `mvn verify` on `components/camel-soap` passes — existing tests 
`SoapToSoapIgnoreTest` and `SoapToSoapDontIgnoreTest` exercise the runtime 
behavior of `ignoreUnmarshalledHeaders`
   - [x] `mvn install` on `core/camel-core-model` and `core/camel-core-reifier` 
succeeds
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to