jasperjiaguo commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r740649780
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/transport/QueryServer.java
##########
@@ -110,7 +119,9 @@ private void attachSSLHandler(SocketChannel ch) {
throw new IllegalArgumentException("Must provide key store path for
secured server");
}
- SslContextBuilder sslContextBuilder =
SslContextBuilder.forServer(TlsUtils.createKeyManagerFactory(_tlsConfig));
+ SslContextBuilder sslContextBuilder = SslContextBuilder
+ .forServer(TlsUtils.createKeyManagerFactory(_tlsConfig))
+ .sslProvider(SslProvider.OPENSSL);
Review comment:
Agreed on the idea of adding native implementation to TLsConfig. Will do.
For creating a streamlined creation method in TlsUtils I think we can start
another PR dedicated for this inssue, so that this PR can be on encryption and
authz.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
