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.