Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/#review132913 --- src/common/http.cpp (line 592)

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/#review132537 --- Ship it! Ship It! - Kapil Arya On May 10, 2016, 4:31 p.m.,

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/ --- (Updated May 10, 2016, 8:31 p.m.) Review request for mesos, Alexander Rojas

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/ --- (Updated May 6, 2016, 9:16 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

> On May 4, 2016, 3:24 a.m., Kapil Arya wrote: > > src/common/http.cpp, line 599 > > > > > > Does non-existing `subject` mean "ANY"? Yep, that's correct. In the local authorizer code

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/ --- (Updated May 5, 2016, 1:38 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/#review131616 --- src/common/http.cpp (line 599)

Review Request 46883: Added authorization callback for '/metrics/snapshot'.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46883/ --- Review request for mesos, Alexander Rojas and Kapil Arya. Bugs: MESOS-5286