----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71666/#review218472 -----------------------------------------------------------
Patch looks great! Reviews applied: [71659, 71660, 71661, 71662, 71663, 71664, 71665, 71666] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers --disable-parallel-test-execution' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Oct. 31, 2019, 1:35 a.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71666/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2019, 1:35 a.m.) > > > Review request for mesos, Benno Evers, Benjamin Mahler, Greg Mann, and Till > Toenshoff. > > > Bugs: MESOS-10010 > https://issues.apache.org/jira/browse/MESOS-10010 > > > Repository: mesos > > > Description > ------- > > This completes a fully functional client-side SSL socket. > > Needs a bit of cleanup and more error handling though. > > > Diffs > ----- > > 3rdparty/libprocess/src/ssl/socket_wrapper.hpp PRE-CREATION > 3rdparty/libprocess/src/ssl/socket_wrapper.cpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/71666/diff/2/ > > > Testing > ------- > > Successfully fetched from a webpage: > ``` > http::URL url = http::URL( > "https", > "www.google.com", > 443); > > Future<http::Response> response = http::get(url); > AWAIT_READY(response); > EXPECT_EQ(http::Status::OK, response->code); > ``` > > Running libprocess-tests yields: > ``` > [ FAILED ] SSLTest.SilentSocket (hangs indefinitely) > [ FAILED ] SSLTest.ValidDowngrade > [ FAILED ] SSLTest.ValidDowngradeEachProtocol > [ FAILED ] SSLTest.ShutdownThenSend > ``` > > > Thanks, > > Joseph Wu > >
