----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42027/#review113481 -----------------------------------------------------------
Ship it! This does not only change 401 to 403 but also removes the "Mesos master" realm from 401 responses -- which should be in line with the behavior expected from the RFC-2617 (is it?). Hence the ticket should be renamed accordingly or these changes should be extracted to another RR. Changing the realm will probably resolve tickets `MESOS-3933` & `MESOS-4197`. - Jan Schlicht On Jan. 8, 2016, 2:48 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42027/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2016, 2:48 p.m.) > > > Review request for mesos, Alexander Rukletsov, Greg Mann, Joerg Schad, Jan > Schlicht, and Till Toenshoff. > > > Bugs: MESOS-4305 > https://issues.apache.org/jira/browse/MESOS-4305 > > > Repository: mesos > > > Description > ------- > > Check MESOS-4305 as to why this change was necesary. > > Changes in the documentation of the endpoints still lacking. > > > Diffs > ----- > > docs/authorization.md a928f1722dc67cd791d78ebbe4591f2e8f2e8f2a > src/master/http.cpp bcafc7aff89659a68352f3876ce6042f8b34bd5d > src/master/quota_handler.cpp 134a93b1d1b6e050aa8a5037ffbec2cc305b0694 > src/tests/master_quota_tests.cpp 776a168254af6fa8a5d87d4580b35d83f2d5909a > src/tests/persistent_volume_endpoints_tests.cpp > f0cce190abc90f0fae84d6c3db20e8215c2d8132 > src/tests/reservation_endpoints_tests.cpp > b8edd6fafedd4c2221a8d19c1ebc71254071a8c7 > src/tests/scheduler_http_api_tests.cpp > 4d23a5a8368e0ed126469fa4a90a889b339ad004 > src/tests/teardown_tests.cpp 97cc89ba168aefff8512f6d1a25c4f7ddf180bae > > Diff: https://reviews.apache.org/r/42027/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
