davsclaus opened a new pull request, #24736: URL: https://github.com/apache/camel/pull/24736
## Summary - Only append `transport.*` options to the Qpid JMS connection URI when the corresponding store location is actually configured (non-null). Previously, all six options were always appended with empty strings when `useSsl=true`, causing Qpid JMS to throw `FileNotFoundException` instead of using the JVM default trust store. - URL-encode transport option values to prevent URI corruption from special characters in store paths or passwords. - Add tests for trust-store-only TLS (server-only TLS, no client key store) and URL encoding of special characters. ## Test plan - [x] `testEnabledSslComponent` — verifies `useSsl=true` with no stores produces `amqps://localhost:5672` (no empty query params) - [x] `testConfiguredSslComponent` — verifies fully configured SSL still works - [x] `testTrustStoreOnlySslComponent` — verifies trust-store-only TLS (common server-only case) - [x] `testSslUrlEncodedPassword` — verifies special chars (`@`, `=`, `&`, `/`) are URL-encoded - [x] Full `camel-amqp` test suite (25 tests pass) _Claude Code on behalf of davsclaus_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) 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]
