From 7dbdf786cc48244943da7dc7884b23763d4eeb31 Mon Sep 17 00:00:00 2001 From: Rohit Yadav Date: Wed, 12 Dec 2012 14:21:38 -0800 Subject: [PATCH] api: FIXME for api access checker adapter Signed-off-by: Rohit Yadav --- server/src/com/cloud/api/ApiServer.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/server/src/com/cloud/api/ApiServer.java b/server/src/com/cloud/api/ApiServer.java index 38bae138786..c7cbaa18d19 100755 --- a/server/src/com/cloud/api/ApiServer.java +++ b/server/src/com/cloud/api/ApiServer.java @@ -770,6 +770,8 @@ public class ApiServer implements HttpRequestHandler { private boolean isCommandAvailable(User user, String commandName) { for(APIAccessChecker apichecker : _apiAccessCheckers) { + // Break on the first adapter that validates + // FIXME: In case of multiple adapters, this may not be the best way of validation. See CLOUDSTACK-618 if (apichecker.canAccessAPI(user, commandName)) return true; }