Skip to content

Add SSL Cassandra support - #1466

Closed
robertdkmarshall wants to merge 1 commit into
apache:masterfrom
instaclustr:master
Closed

Add SSL Cassandra support#1466
robertdkmarshall wants to merge 1 commit into
apache:masterfrom
instaclustr:master

Conversation

@robertdkmarshall

Copy link
Copy Markdown

What is this PR for?

The Cassandra Interpreter does not support talking to clusters that use SSL/client to node encryption. It does not have the properties needed to configure the SSL Context. This PR adds the properties to the driver config and sets up the ssl options when they are requested.

What type of PR is it?

Improvement

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1501

How should this be tested?

Using CQL interpreter connect to a Cassandra cluster that uses client to node encryption.

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this need documentation? Yes. Should add the new properties to the list (cassandra.ssl.enabled, cassandra.ssl.truststore.path and cassandra.ssl.truststore.password).

@Leemoonsoo

Copy link
Copy Markdown
Member

\cc @doanduyhai

@doanduyhai

Copy link
Copy Markdown
Contributor

Ping @robertdkmarshall, any new from my code review ?

@doanduyhai

Copy link
Copy Markdown
Contributor

Without feedback I'll push a new PR with suggested code refactor & merge it in a few weeks @robertdkmarshall,

@jcam3ron

Copy link
Copy Markdown

Any update on whether this might be merged soon?

@doanduyhai

Copy link
Copy Markdown
Contributor

Let me redo the PR and merge with the above remarks included

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@robertdkmarshall@Leemoonsoo@doanduyhai@jcam3ron