You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2022/08/01 02:07:29 UTC

[GitHub] [rocketmq-mqtt] ChangingFond commented on a diff in pull request #123: use openssl provider instead of jdk provider to improve performance

ChangingFond commented on code in PR #123:
URL: https://github.com/apache/rocketmq-mqtt/pull/123#discussion_r934085974


##########
mqtt-cs/src/main/java/org/apache/rocketmq/mqtt/cs/protocol/ssl/SslFactory.java:
##########
@@ -59,7 +59,7 @@ private void initSslContext() {
             InputStream keyStream = new ClassPathResource(KEY_FILE_NAME).getInputStream();
             SslContextBuilder contextBuilder = SslContextBuilder.forServer(certStream, keyStream);
             contextBuilder.clientAuth(ClientAuth.OPTIONAL);
-            contextBuilder.sslProvider(SslProvider.JDK);
+            contextBuilder.sslProvider(SslProvider.OPENSSL);

Review Comment:
   thx, this fix is committed



-- 
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: dev-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org