----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61189/#review181716 -----------------------------------------------------------
src/master/master.hpp Lines 1835 (patched) <https://reviews.apache.org/r/61189/#comment257364> Does `Subscriber` need to store the authorizer? Can it just do `master->authorizer`? src/master/master.cpp Lines 9586 (patched) <https://reviews.apache.org/r/61189/#comment258210> Perhaps we could make this a member function of `Subscriber` and eliminate the `subscriber` parameter? Then you could do `subscriber->send(...)` here. - Greg Mann On July 27, 2017, 6:25 p.m., Quinn Leng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61189/ > ----------------------------------------------------------- > > (Updated July 27, 2017, 6:25 p.m.) > > > Review request for mesos, Anand Mazumdar, Alexander Rojas, and Greg Mann. > > > Bugs: MESOS-7785 > https://issues.apache.org/jira/browse/MESOS-7785 > > > Repository: mesos > > > Description > ------- > > Added authorization filtering for V1 streaming events, the > subscriber should only receive events that are authorized > based on their principal and ACLs. > > > Diffs > ----- > > src/master/http.cpp 9df086c417a9392f62d600c7a6486be0a1cf7e70 > src/master/master.hpp 84465af782d4024f22463d981ef9d0ef7827d043 > src/master/master.cpp e12c997dad04f8a4ddb47a993a84b2b05c9e2f32 > src/tests/api_tests.cpp f22ca28c819712d8797e0c0c69dc1ebf1fe5ac1f > > > Diff: https://reviews.apache.org/r/61189/diff/2/ > > > Testing > ------- > > make check > GLOG_v=2 ./bin/mesos-tests.sh > --gtest_filter="ContentType/MasterAPITest.EventsAuthorizationFiltering*" > --verbose --gtest_repeat=100 --gtest_break_on_failure > > > Thanks, > > Quinn Leng > >
