----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38399/#review99376 -----------------------------------------------------------
I think that you also need some test cases for src/tests/authorization_tests.cpp. docs/authentication.md (line 14) <https://reviews.apache.org/r/38399/#comment156255> There are four endpoints: /maintenance/schedule /maintenance/status /machine/down /machine/up Is it possible to list them here? I see that you did not handle the ACL for /maintenance/status, does this needed? Thanks. - Guangya Liu On Sept. 17, 2015, 8:31 a.m., Zhiwei Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38399/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 8:31 a.m.) > > > Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph > Wu. > > > Bugs: mesos-2222 > https://issues.apache.org/jira/browse/mesos-2222 > > > Repository: mesos > > > Description > ------- > > Add ACLs for the maintenance HTTP endpoints > > > Diffs > ----- > > docs/authentication.md 1c22c5416caf66b28238fc181a255e51ed16d867 > docs/authorization.md 2a5e74782754e446184a297f91112e9f94077896 > include/mesos/authorizer/authorizer.hpp > d667a52f90f970a313580446a5a006cec4b5e25b > include/mesos/authorizer/authorizer.proto > 86bbb45f9d91b4098a262e3e50a793f3bb39497e > src/authorizer/local/authorizer.hpp > 32de102fd588f029882ef2222121ca83a7410c65 > src/authorizer/local/authorizer.cpp > 6d7da87731a438c2180cf91003e09d4aa5a1c773 > src/master/flags.cpp 80879611fbcfd764c9fc8f60a31613a9c8fc2364 > src/master/http.cpp f7ce9aa56b453c1d37b99dd836d956ab292ab62e > src/master/master.hpp d48ef7c0da8978a5e02e69e055ff010585b20ceb > src/tests/master_maintenance_tests.cpp > 44785057f129a3e6a69f399f7d6db59d9d5c2e91 > src/tests/mesos.hpp 3db97aca921c9216d90384e1eb17030849516454 > > Diff: https://reviews.apache.org/r/38399/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Zhiwei Chen > >
