This is an automated email from the ASF dual-hosted git repository.
arina pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git
The following commit(s) were added to refs/heads/master by this push:
new 370bd2a DRILL-7644: Log SSL protocol version at Drill start up
370bd2a is described below
commit 370bd2ad99a469056841c0fb35169a401827a9a9
Author: Igor Guzenko <[email protected]>
AuthorDate: Tue Mar 17 17:58:06 2020 +0200
DRILL-7644: Log SSL protocol version at Drill start up
---
.../src/main/java/org/apache/drill/exec/rpc/user/UserServer.java | 1 +
.../apache/drill/exec/server/rest/ssl/SslContextFactoryConfigurator.java | 1 +
2 files changed, 2 insertions(+)
diff --git
a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java
b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java
index 1c2e2e2..276758e 100644
---
a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java
+++
b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java
@@ -137,6 +137,7 @@ public class UserServer extends BasicServer<RpcType,
BitToUserConnection> {
.initializeSSLContext(true)
.validateKeyStore(true)
.build();
+ logger.info("Rpc server configured to use TLS protocol '{}'",
sslConfig.getProtocol());
} catch (DrillException e) {
throw new DrillbitStartupException(e.getMessage(), e.getCause());
}
diff --git
a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/ssl/SslContextFactoryConfigurator.java
b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/ssl/SslContextFactoryConfigurator.java
index a5aa541..62a740b 100644
---
a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/ssl/SslContextFactoryConfigurator.java
+++
b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/ssl/SslContextFactoryConfigurator.java
@@ -95,6 +95,7 @@ public class SslContextFactoryConfigurator {
}
sslFactory.setProtocol(sslConf.getProtocol());
sslFactory.setIncludeProtocols(sslConf.getProtocol());
+ logger.info("Web server configured to use TLS protocol '{}'",
sslConf.getProtocol());
if
(config.hasPath(ExecConstants.HTTP_JETTY_SSL_CONTEXT_FACTORY_OPTIONS_PREFIX)) {
setStringIfPresent(ExecConstants.HTTP_JETTY_SERVER_SSL_CONTEXT_FACTORY_CERT_ALIAS,
sslFactory::setCertAlias);
setStringIfPresent(ExecConstants.HTTP_JETTY_SERVER_SSL_CONTEXT_FACTORY_CRL_PATH,
sslFactory::setCrlPath);