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