luocooong commented on pull request #2505: URL: https://github.com/apache/drill/pull/2505#issuecomment-1079879536
@rymarm Thanks for the update. If we update the default SSL configuration, we also may have to update the default options and unit tests for the codebase. ``` drill-module.conf - drill-java-exec/src/main/resources drill-override-example.conf - distribution/src/main/resources ExecConstants.java - drill-java-exec/src/main/java/org/apache/drill/exec SSLConfig.java - drill-java-exec/src/main/java/org/apache/drill/exec/ssl SslContextFactoryConfiguratorTest.java - drill-java-exec/src/test/java/org/apache/drill/exec/server/rest/ssl TestUserBitSSL.java - drill-java-exec/src/test/java/org/apache/drill/exec/rpc/user/security TestUserBitSSLServer.java - drill-java-exec/src/test/java/org/apache/drill/exec/rpc/user/security ``` There are also associated documents [Configuring SSL/TLS for Encryption](https://github.com/apache/drill-site/blob/master/_docs/en/configure-drill/securing-drill/081-configuring-ssl-tls-for-encryption.md) to be update. Fork the `drill-site` and create new pull request for the markdown file. -- 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-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org