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 "