mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.
Date Mon, 25 Apr 2016 09:52:10 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46319/
-----------------------------------------------------------

(Updated April 25, 2016, 11:52 a.m.)


Review request for mesos, Adam B, Alexander Rojas, and Jan Schlicht.


Changes
-------

Rebased onto Jan's last changes. Expected to fail until he fixes his `authorizeEndpoint` to
support agent paths containing multiple `/`.


Bugs: MESOS-5164
    https://issues.apache.org/jira/browse/MESOS-5164


Repository: mesos


Description
-------

Added authorization to agents' `/monitor/statistics` endpoints.


Diffs (updated)
-----

  src/slave/http.cpp 537736d1fe42e8150bad91326299ef9a17041a8e 
  src/slave/slave.hpp 20a4bcd0bb9dad06ea81fc4ad9b2fa462c69d2c5 
  src/tests/slave_authorization_tests.cpp PRE-CREATION 

Diff: https://reviews.apache.org/r/46319/diff/


Testing
-------

make check (OS X, clang w/o optimization)


Thanks,

Benjamin Bannier


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message