You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2017/04/18 22:39:45 UTC

mesos git commit: Added a TODO for implicit scheduler authorization.

Repository: mesos
Updated Branches:
  refs/heads/master b939f185f -> f253811c8


Added a TODO for implicit scheduler authorization.

This patch adds a TODO to the master's HTTP scheduler API related
to MESOS-7399.

Review: https://reviews.apache.org/r/58507/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/f253811c
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/f253811c
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/f253811c

Branch: refs/heads/master
Commit: f253811c8cbaa4c4fc9c2b05b68e2b8161d35aa4
Parents: b939f18
Author: Greg Mann <gr...@mesosphere.io>
Authored: Tue Apr 18 15:39:23 2017 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Tue Apr 18 15:39:23 2017 -0700

----------------------------------------------------------------------
 src/master/http.cpp | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/f253811c/src/master/http.cpp
----------------------------------------------------------------------
diff --git a/src/master/http.cpp b/src/master/http.cpp
index 0e84d36..5aae528 100644
--- a/src/master/http.cpp
+++ b/src/master/http.cpp
@@ -882,6 +882,8 @@ Future<Response> Master::Http::scheduler(
     return BadRequest("Framework cannot be found");
   }
 
+  // TODO(greggomann): Move this implicit scheduler authorization
+  // into the authorizer. See MESOS-7399.
   if (principal.isSome() && principal != framework->info.principal()) {
     return BadRequest(
         "Authenticated principal '" + stringify(principal.get()) + "' does not "