----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36656/#review92638 -----------------------------------------------------------
Ship it! + Comments from Artem and BenH. 3rdparty/libprocess/src/openssl.cpp (line 327) <https://reviews.apache.org/r/36656/#comment146860> Saw this while grepping for `SSLv2`. We still need to use this rather than `TLS_method` currently because of support for older OpenSSL, correct? Just wondering since we no longer support `SSLv2`. OpenSSL documentation [1] mentions that `SSLv23_method` has been replaced with `TLS_method` which supports `SSLv3`, `TLSv1`, `TLSv1.1` and `TLSv1.2`. It also mentioned that `TLS_method` was introduced in OpenSSL 1.1.0. [1] https://www.openssl.org/docs/ssl/TLSv1_1_server_method.html 3rdparty/libprocess/src/openssl.cpp (lines 478 - 484) <https://reviews.apache.org/r/36656/#comment146859> Saw this while grepping for `SSLv2`. I feel like `SSL_CTX_clear_options(ctx, SSL_CTX_get_options(ctx));` would be a concise way to do this? It would also continue to completely clear the options, even if new protocols were to be introduced into OpenSSL. - Michael Park On July 21, 2015, 10:51 p.m., Joris Van Remoortere wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36656/ > ----------------------------------------------------------- > > (Updated July 21, 2015, 10:51 p.m.) > > > Review request for mesos, Benjamin Hindman and Michael Park. > > > Bugs: MESOS-3121 > https://issues.apache.org/jira/browse/MESOS-3121 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > 3rdparty/libprocess/src/openssl.hpp > 3f8d351589f8bb26c886da12c53b5e02a242376a > 3rdparty/libprocess/src/openssl.cpp > 6aa2a4db8d64011d0fde6ff0cf4b144c41949d39 > 3rdparty/libprocess/src/tests/ssl_tests.cpp > 2fe50601615b0bee57bd3e05dc9c932f93ca7477 > > Diff: https://reviews.apache.org/r/36656/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Joris Van Remoortere > >