[
https://issues.apache.org/jira/browse/DRILL-5431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16175868#comment-16175868
]
ASF GitHub Bot commented on DRILL-5431:
---------------------------------------
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/950#discussion_r140400292
--- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicServer.java
---
@@ -105,6 +109,25 @@ protected void initChannel(SocketChannel ch) throws
Exception {
// }
}
+ // Adds a SSL handler if enabled. Required only for client and server
communications, so
+ // a real implementation is only available for UserServer
+ protected void setupSSL(ChannelPipeline pipe) {
+ // Do nothing
+ }
+
+ protected boolean isSslEnabled() {
+ return false;
+ }
+
+ // Save the SslChannel after the SSL handshake so it can be closed later
+ public void setSslChannel(Channel c) {
+ return;
+ }
+
+ protected void closeSSL() {
+ return;
+ }
--- End diff --
redundant `return` statement in both `closeSSL` and `setSSLChannel`.
> 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: Sudheesh Katkam
>
> 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)
