Improve authenticator mgt to store image and description.

wso2 · Jan 20, 2025 · 7b1370e · 7b1370e
1 parent e6e74dd
commit 7b1370e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/...cution/src/main/java/org/wso2/carbon/identity/action/execution/ActionExecutorService.java b/...cution/src/main/java/org/wso2/carbon/identity/action/execution/ActionExecutorService.java
@@ -52,7 +52,7 @@ ActionExecutionStatus execute(ActionType actionType, Map<String, Object> eventCo
             ActionExecutionException;
 
     /**
-     * Resolve the action by given the action id and execute them.
+     * Resolve the action from given action id and execute it.
      *
      * @param actionType    Action Type.
      * @param actionId      The action Id of the action that need to be executed.

diff --git a/...c/main/java/org/wso2/carbon/identity/action/execution/impl/ActionExecutorServiceImpl.java b/...c/main/java/org/wso2/carbon/identity/action/execution/impl/ActionExecutorServiceImpl.java
@@ -126,7 +126,7 @@ public ActionExecutionStatus<?> execute(ActionType actionType, Map<String, Objec
     }
 
     /**
-     * Resolve the action by given the action id list and execute them.
+     * Resolve the action from given action id and execute it.
      *
      * @param actionType    Action Type.
      * @param actionId      The action Id of the action that need to be executed.
@@ -139,15 +139,15 @@ public ActionExecutionStatus<?> execute(ActionType actionType, String actionId,
                                             Map<String, Object> eventContext, String tenantDomain)
             throws ActionExecutionException {
 
-        if (actionId == null) {
-            throw new ActionExecutionException("No action Ids found for action type: " + actionType.name());
+        if (actionId.isEmpty()) {
+            throw new ActionExecutionException("Action Id cannot be empty.");
         }
 
         Action action = getActionByActionId(actionType, actionId, tenantDomain);
         try {
             return execute(action, eventContext);
         } catch (ActionExecutionRuntimeException e) {
-            LOG.debug("Skip executing actions for action type: " + actionType.name(), e);
+            LOG.debug("Skip executing action for action type: " + actionType.name(), e);
             // Skip executing actions when no action available is considered as action execution being successful.
             return new SuccessStatus.Builder().setResponseContext(eventContext).build();
         }