davsclaus opened a new pull request, #24737: URL: https://github.com/apache/camel/pull/24737
## Summary Fixes four medium-severity findings from the camel-jms/camel-amqp code review ([CAMEL-24078](https://issues.apache.org/jira/browse/CAMEL-24078)): - **M3**: Endpoint-level `includeAllJMSXProperties=true` was silently ignored because the header filter strategy was constructed before URI options were applied. Now resets the cached `JmsHeaderFilterStrategy` when `includeAllJMSXProperties` changes. - **M4**: Component-level `messageListenerContainerFactory` was silently ignored because `consumerType` was not switched to `Custom`. Aligned with the endpoint-level setter that already does this. - **M6**: `asyncStartListener=true` defeated `autoStartup=false` due to a race — `initialized` was set before the async task ran, bypassing the auto-startup check. Moved the flag assignment into `prepareAndStartListenerContainer()`. - **M9**: Setting `username`/`password` on the AMQP component clobbered an explicitly configured `connectionFactory` with a new `localhost:5672` factory. Guarded factory creation with a null check. ## Test plan - [x] `mvn test` in `components/camel-jms` — all tests pass - [x] `mvn test` in `components/camel-amqp` — all tests pass - [ ] 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]
