You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bh...@apache.org on 2013/01/15 04:01:44 UTC
[7/8] git commit: ApiServer: Process through chain of api access
checker, on failure it will throw exception
ApiServer: Process through chain of api access checker, on failure it will throw exception
Signed-off-by: Rohit Yadav <bh...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/8f27c711
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/8f27c711
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/8f27c711
Branch: refs/heads/master
Commit: 8f27c711e58e447409fb1b496dfeab9cf4cd7d63
Parents: 0a31945
Author: Rohit Yadav <bh...@apache.org>
Authored: Mon Jan 14 18:56:02 2013 -0800
Committer: Rohit Yadav <bh...@apache.org>
Committed: Mon Jan 14 19:01:26 2013 -0800
----------------------------------------------------------------------
server/src/com/cloud/api/ApiServer.java | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/8f27c711/server/src/com/cloud/api/ApiServer.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/api/ApiServer.java b/server/src/com/cloud/api/ApiServer.java
index c8511b2..4d60215 100755
--- a/server/src/com/cloud/api/ApiServer.java
+++ b/server/src/com/cloud/api/ApiServer.java
@@ -786,9 +786,7 @@ public class ApiServer implements HttpRequestHandler {
}
for (APIChecker apiChecker : _apiAccessCheckers) {
- // Fail the checking if any checker fails to verify
- if (!apiChecker.checkAccess(user, commandName))
- return false;
+ apiChecker.checkAccess(user, commandName);
}
return true;
}