davsclaus opened a new pull request, #24735: URL: https://github.com/apache/camel/pull/24735
## Summary - Wire `configuration.getBrowseLimit()` into `JmsQueueEndpoint`, `ActiveMQQueueEndpoint` (camel-activemq), and `ActiveMQQueueEndpoint` (camel-activemq6) via `doInit()`, so the `browseLimit` URI option actually takes effect - The generated configurer routes `browseLimit` to `JmsConfiguration.setBrowseLimit()`, but `JmsQueueEndpoint` uses its own `maximumBrowseSize` field which was never populated from the config — making the option dead code since commit caa51c5da0a8 - Add test for the `browseLimit` URI option in `BrowsableQueueTest` ## Test plan - [x] `BrowsableQueueTest` passes (4 tests including new `testSendMessagesThenBrowseQueueWithBrowseLimit`) - [ ] 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]
