----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68104/#review206684 -----------------------------------------------------------
Ship it! src/slave/http.cpp Lines 1831-1833 (patched) <https://reviews.apache.org/r/68104/#comment289769> It seems we can hoist this outside the loop. However we might want to consider having finer-grained ACLs in the future so I'm fine with keeping it as is. - Chun-Hung Hsiao On July 31, 2018, 10:19 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68104/ > ----------------------------------------------------------- > > (Updated July 31, 2018, 10:19 a.m.) > > > Review request for mesos, Chun-Hung Hsiao and Jan Schlicht. > > > Bugs: MESOS-8314 > https://issues.apache.org/jira/browse/MESOS-8314 > > > Repository: mesos > > > Description > ------- > > Implemented authorization for agent `GET_RESOURCE_PROVIDER` calls. > > > Diffs > ----- > > src/slave/http.cpp ab5864d9fd2fde478ed7da2ca7ed8abedc72c7c5 > src/tests/api_tests.cpp 182622a62d350ebefc891a385de3f2d35a7c0243 > > > Diff: https://reviews.apache.org/r/68104/diff/2/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >