RockteMQ-AI commented on code in PR #765:
URL: https://github.com/apache/rocketmq-spring/pull/765#discussion_r3867822007
##########
rocketmq-v5-client-spring-boot/src/main/java/org/apache/rocketmq/client/annotation/RocketMQMessageListener.java:
##########
@@ -58,6 +58,11 @@
*/
String tag() default TAG_PLACEHOLDER;
+ /**
+ * Enable or disable the use of Secure Sockets Layer (SSL) for network
transport.
+ */
+ boolean sslEnabled() default true;
Review Comment:
**[Warning]** `sslEnabled` defaults to `true`. This may be a **breaking
change** for existing users who do not have SSL configured — their listeners
would suddenly fail to connect after upgrading.
Consider defaulting to `false` to preserve backward compatibility:
```java
boolean sslEnabled() default false;
```
Users who need SSL can explicitly opt in via
`@RocketMQMessageListener(sslEnabled = true)`.
--
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]
