----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71666/#review218948 -----------------------------------------------------------
3rdparty/libprocess/src/ssl/socket_wrapper.hpp Lines 94 (patched) <https://reviews.apache.org/r/71666/#comment306923> In this case, we don't really need an actor context, since there isn't any actor state associated with the compute thread. We really just want some context (any context) to dispatch the SSL-related functions onto, right? It would make a bit more sense to me to dispatch these functions without specifying an actor, so that libprocess can run them wherever it pleases. We could consider updating `loop()` to dispatch in all cases, even when no pid is specified. However, I do wonder if we're unknowingly depending on the existing behavior somewhere. In any case, changing loop to always `dispatch()` the iterate and body seems more desirable to me? However, the `loop()` calls below aren't strictly necessary I think. We could accomplish the same thing with dispatches and chained continuations, so we could also just use `dispatch()` directly instead of `loop()`, that might be the simplest thing to do. WDYT? - Greg Mann On Nov. 20, 2019, 12:29 a.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71666/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2019, 12:29 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/6/ > > > Testing > ------- > > ``` > cmake --build . --target libprocess-tests > libprocess-tests > ``` > > Running libprocess-tests yields: > ``` > [ FAILED ] SSLTest.ValidDowngrade > [ FAILED ] SSLTest.ValidDowngradeEachProtocol > ``` > > > Thanks, > > Joseph Wu > >
