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

Jason Brown commented on CASSANDRA-4239:
----------------------------------------

Hey guys,

Using a TTransportFactory derived class comes from CASSANDRA-4609, and that's 
what both cli and stress use for a pluggable thrift transport. The problem with 
using a ctor, which I first tried when implementing this (to alleviate 
depending on stress's Session and cli's CLiSessionState, respectively), was 
that you end up needing to special case the TTransportFactory implementation to 
make sure you get all the needed values to the ctor. As this scheme for cli and 
stress was meant to be pluggable, having that special casing inside of the c* 
code seemed contrary to making it abstract.

I think moving stress's SSLTransportFactory class to different package is 
reasonable.

                
> Support Thrift SSL socket
> -------------------------
>
>                 Key: CASSANDRA-4239
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4239
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: API
>            Reporter: Jonathan Ellis
>            Assignee: Jason Brown
>            Priority: Minor
>             Fix For: 1.2.1
>
>         Attachments: 
> 0001-CASSANDRA-4239-Support-Thrift-SSL-socket-both-to-the.patch, 
> 0001-CASSANDRA-4239-v3.patch, 0002-CASSANDRA-4239-Support-Thrift-SSL.patch
>
>
> Thrift has supported SSL encryption for a while now (THRIFT-106); we should 
> allow configuring that in cassandra.yaml

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to