[ https://issues.apache.org/jira/browse/DRILL-5431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16191362#comment-16191362 ]
ASF GitHub Bot commented on DRILL-5431: --------------------------------------- Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r142681811 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/TestSSLConfig.java --- @@ -49,12 +64,19 @@ public void testMissingKeystorePassword() throws Exception { ConfigBuilder config = new ConfigBuilder(); config.put(ExecConstants.HTTP_KEYSTORE_PATH, "/root"); config.put(ExecConstants.HTTP_KEYSTORE_PASSWORD, ""); + config.put(ExecConstants.SSL_USE_HADOOP_CONF, false); + config.put(ExecConstants.USER_SSL_ENABLED, true); try { - SSLConfig sslv = new SSLConfig(config.build()); + SSLConfig sslv = new SSLConfigBuilder() + .config(config.build()) + .mode(SSLFactory.Mode.SERVER) + .initializeSSLContext(false) + .validateKeyStore(true) + .build(); fail(); //Expected } catch (Exception e) { - assertTrue(e instanceof DrillException); + --- End diff -- So what is test is actually testing? Since we just fail and ignore the exception. > Support SSL > ----------- > > Key: DRILL-5431 > URL: https://issues.apache.org/jira/browse/DRILL-5431 > Project: Apache Drill > Issue Type: New Feature > Components: Client - Java, Client - ODBC > Reporter: Sudheesh Katkam > Assignee: Parth Chandra > Labels: doc-impacting > Fix For: 1.12.0 > > > Support SSL between Drillbit and JDBC/ODBC drivers. Drill already supports > HTTPS for web traffic. -- This message was sent by Atlassian JIRA (v6.4.14#64029)