You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by "cgivre (via GitHub)" <gi...@apache.org> on 2023/04/24 16:58:29 UTC

[PR] DRILL-8385: Add support for disabling SSL certificate verification in the Elasticsearch plugin (drill)

cgivre opened a new pull request, #2795:
URL: https://github.com/apache/drill/pull/2795

   # [DRILL-8385](https://issues.apache.org/jira/browse/DRILL-8385): Add support for disabling SSL certificate verification in the Elasticsearch plugin
   
   ## Description
   Adds a config option to the ElasticSearch plugin, `disableSSLVerification` which disables SSL verification.  Defaults to `false`.
   
   ## Documentation
   See above. 
   
   ## Testing
   Updated unit tests.


-- 
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-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] DRILL-8385: Add support for disabling SSL certificate verification in the Elasticsearch plugin (drill)

Posted by "jnturton (via GitHub)" <gi...@apache.org>.
jnturton commented on PR #2795:
URL: https://github.com/apache/drill/pull/2795#issuecomment-1531244090

   Since Drill master is based on Calcite main this should be usable in master as soon as it's merged.


-- 
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-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] DRILL-8385: Add support for disabling SSL certificate verification in the Elasticsearch plugin (drill)

Posted by "cgivre (via GitHub)" <gi...@apache.org>.
cgivre commented on PR #2795:
URL: https://github.com/apache/drill/pull/2795#issuecomment-1520523927

   Note to reviewers, this PR will have no effect until [CALCITE-5671 ](https://github.com/apache/calcite/pull/3174) is merged.


-- 
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-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] DRILL-8385: Add support for disabling SSL certificate verification in the Elasticsearch plugin (drill)

Posted by "cgivre (via GitHub)" <gi...@apache.org>.
cgivre commented on PR #2795:
URL: https://github.com/apache/drill/pull/2795#issuecomment-1529744787

   @jnturton Would you mind doing a review on this?  The Calcite work is done and has been accepted and the PR is closed.


-- 
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-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] DRILL-8385: Add support for disabling SSL certificate verification in the Elasticsearch plugin (drill)

Posted by "cgivre (via GitHub)" <gi...@apache.org>.
cgivre merged PR #2795:
URL: https://github.com/apache/drill/pull/2795


-- 
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-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org