fix boolean param value for custom action

Signed-off-by: Abhishek Kumar <[email protected]>

Author: shwstppr

api/src/main/java/org/apache/cloudstack/extension/ExtensionCustomAction.java

@@ -290,7 +290,10 @@ public Object validatedValue(String value) {
             try {
                 switch (type) {
                     case BOOLEAN:
-                        return Arrays.asList("true", "false").contains(value);
+                        if (!Arrays.asList("true", "false").contains(value)) {
+                            throw new IllegalArgumentException();
+                        }
+                        return Boolean.parseBoolean(value);
                     case DATE:
                         return DateUtil.parseTZDateString(value);
                     case NUMBER:

framework/extensions/src/main/java/org/apache/cloudstack/framework/extensions/manager/ExtensionsManagerImpl.java

@@ -1347,7 +1347,8 @@ public CustomActionResultResponse runCustomAction(RunCustomActionCmd cmd) {
         runCustomActionCommand.setExternalDetails(externalDetails);
         runCustomActionCommand.setWait(customActionVO.getTimeout());
         try {
-            logger.info("Running custom action: {}", GsonHelper.getGson().toJson(runCustomActionCommand));
+            logger.info("Running custom action: {} with {} parameters", actionName,
+                    (parameters != null ? parameters.keySet().size() : 0));
             Answer answer = agentMgr.send(hostId, runCustomActionCommand);
             if (!(answer instanceof RunCustomActionAnswer)) {
                 logger.error("Unexpected answer [{}] received for {}", answer.getClass().getSimpleName(),