zhangshenghang opened a new pull request, #11576: URL: https://github.com/apache/seatunnel/pull/11576
## Purpose Fixes #9923. This PR preserves the existing `url` option, accepts `uri` as its backward-compatible alias, and adds `ssl` and `passive` options. `ssl = true` enables TLS for host-and-port connections. `passive = true` verifies an existing queue instead of declaring it, so consumer accounts without declaration permission can connect safely. ## Compatibility Existing configurations keep their current behavior: active queue declaration and plaintext host-and-port connections remain the defaults. AMQPS URIs continue to work through `url`. ## Tests - Added unit coverage for the `uri`, `ssl`, and `passive` configuration values. - Added unit coverage for passive and active queue declaration behavior. - `JAVA_HOME=$(/usr/libexec/java_home -v 17) ./mvnw -pl seatunnel-connectors-v2/connector-rabbitmq spotless:apply` - `JAVA_HOME=$(/usr/libexec/java_home -v 17) ./mvnw -q -pl seatunnel-connectors-v2/connector-rabbitmq test -Dtest=RabbitmqFactoryTest,RabbitmqClientTest` - `JAVA_HOME=$(/usr/libexec/java_home -v 17) ./mvnw -q -DskipTests verify` The full `./mvnw -q test` suite is blocked before this connector by the shared `seatunnel-config-shade` module missing shaded Typesafe Config classes. -- 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]
