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

Hudson commented on THRIFT-2258:
--------------------------------

SUCCESS: Integrated in Thrift #1053 (See 
[https://builds.apache.org/job/Thrift/1053/])
Revert "THRIFT-2258 cpp: Add TLS v1.1/1.2 support to TSSLSocket.cpp" (jfarrell: 
rev db536cf6bb7a561ca83c7f4b8c1c7fd1fed00375)
* lib/cpp/src/thrift/transport/TSSLSocket.h
* lib/cpp/src/thrift/transport/TSSLSocket.cpp


> Add TLS v1.1/1.2 support to TSSLSocket.cpp
> ------------------------------------------
>
>                 Key: THRIFT-2258
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2258
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>    Affects Versions: 0.9.1
>            Reporter: Chris Stylianou
>            Assignee: Roger Meier
>              Labels: c++, ssl, thrift, tls
>         Attachments: thrift-2258.patch
>
>
> At the moment TSSLSocket.cpp hard-codes the SSL/TLS protocol to TLSv1.0, 
> which does not allow any other protocols to be used instead (SSL v3, TLS 
> v1.0, v1.1, v1.2, *ignores SSLv2 as horribly insecure*).
> Could a method be provided on the TSSLSocketFactory to set the required 
> protocol (like how there is already a cipher() function available), so that 
> when SSL_CTX_new, it is called with the specified SSL/TLS protocol.
> Sorry to label this as a bug, but being unable to select the highest availabe 
> security protocol for communication is a bug in my eyes. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to