Revolyssup commented on code in PR #12465:
URL: https://github.com/apache/apisix/pull/12465#discussion_r2253920478


##########
apisix/plugins/workflow.lua:
##########
@@ -157,5 +170,18 @@ function _M.access(conf, ctx)
     end
 end
 
+function _M.log(conf, ctx)
+    for idx, rule in ipairs(conf.rules) do
+        local match_result = ctx._workflow_cache[idx]
+        if match_result then
+            -- only one action is currently supported
+            local action = rule.actions[1]
+            local log_handler = support_action[action[1]].log_handler
+            if log_handler then

Review Comment:
   Not all plugins might pass log_handler callback. This handler's action only 
needs to be run when log_handler is passed. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to