----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60917/#review180773 -----------------------------------------------------------
Ship it! LGTM. src/log/network.hpp Line 247 (original), 248-250 (patched) <https://reviews.apache.org/r/60917/#comment255997> I'd prefer this comment to be a `NOTE: ...` :) - Joseph Wu On July 17, 2017, noon, Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60917/ > ----------------------------------------------------------- > > (Updated July 17, 2017, noon) > > > Review request for mesos and Joseph Wu. > > > Bugs: MESOS-7753 > https://issues.apache.org/jira/browse/MESOS-7753 > > > Repository: mesos > > > Description > ------- > > Previously an empty UPID `@0.0.0.0:0` was used which could result in > the message being dropped due to certain security check (MESOS-7401). > > > Diffs > ----- > > src/log/network.hpp 0afcfae27394527f7c957e31014cc9a32a4bf63b > > > Diff: https://reviews.apache.org/r/60917/diff/1/ > > > Testing > ------- > > make check. > > > Thanks, > > Jiang Yan Xu > >
