[ 
https://issues.apache.org/jira/browse/DRILL-5431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177168#comment-16177168
 ] 

ASF GitHub Bot commented on DRILL-5431:
---------------------------------------

Github user parthchandra commented on a diff in the pull request:

    https://github.com/apache/drill/pull/950#discussion_r140601194
  
    --- Diff: contrib/native/client/src/include/drill/common.hpp ---
    @@ -163,9 +170,13 @@ typedef enum{
     #define USERPROP_USERNAME "userName"
     #define USERPROP_PASSWORD "password"
     #define USERPROP_SCHEMA   "schema"
    -#define USERPROP_USESSL   "useSSL"        // Not implemented yet
    -#define USERPROP_FILEPATH "pemLocation"   // Not implemented yet
    -#define USERPROP_FILENAME "pemFile"       // Not implemented yet
    +#define USERPROP_USESSL   "enableTLS"
    +#define USERPROP_TLSPROTOCOL "TLSProtocol" //TLS version
    +#define USERPROP_CERTFILEPATH "certFilePath" // pem file path and name
    +#define USERPROP_CERTPASSWORD "certPassword" // Password for certificate 
file
    --- End diff --
    
    Commented it out. I'm leaving it there so it can be used when we implement 
support for trust stores which may require passwords. 


> 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
>
> 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)

Reply via email to