You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2015/12/14 16:00:05 UTC

[2/2] mesos git commit: Fixed a bug in authentication endpoint removal during Process cleanup.

Fixed a bug in authentication endpoint removal during Process cleanup.

When a process is destroyed, it unregisters all of its endpoints from
authentication. However the paths were malformed and the endpoints
were left inside the authentication router.

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


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

Branch: refs/heads/master
Commit: dc6cde747eec94ba35aae5f5587d80338c000068
Parents: 1d4f50a
Author: Alexander Rojas <al...@mesosphere.io>
Authored: Mon Dec 14 08:50:32 2015 -0600
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Mon Dec 14 08:50:32 2015 -0600

----------------------------------------------------------------------
 3rdparty/libprocess/src/process.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/dc6cde74/3rdparty/libprocess/src/process.cpp
----------------------------------------------------------------------
diff --git a/3rdparty/libprocess/src/process.cpp b/3rdparty/libprocess/src/process.cpp
index af3cefb..cff635e 100644
--- a/3rdparty/libprocess/src/process.cpp
+++ b/3rdparty/libprocess/src/process.cpp
@@ -2711,7 +2711,8 @@ void ProcessManager::cleanup(ProcessBase* process)
   // Remove all routes from the authentication router, we don't
   // need to wait for these operations to complete.
   foreachkey (const string& endpoint, process->handlers.http) {
-    authentication_router->removeEndpoint('/' + process->self().id + endpoint);
+    authentication_router->removeEndpoint(
+        '/' + process->self().id + '/' + endpoint);
   }
 
   // Possible gate non-libprocess threads are waiting at.