From d3779a32190222e6d56d8562f9733599b05bf49b Mon Sep 17 00:00:00 2001
From: "Zaiming (Stone) Shi" <zmstone@gmail.com>
Date: Tue, 26 Sep 2023 12:08:07 +0200
Subject: [PATCH] fix(audit): make 'emqx eval' command auditable

---
 bin/nodetool | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/bin/nodetool b/bin/nodetool
index d00d14a12..ab2210aa5 100755
--- a/bin/nodetool
+++ b/bin/nodetool
@@ -140,11 +140,12 @@ do(Args) ->
                     io:format("~p\n", [Other])
             end;
         ["eval" | ListOfArgs] ->
+            % parse args locally in the remsh node
             Parsed = parse_eval_args(ListOfArgs),
             % and evaluate it on the remote node
-            case rpc:call(TargetNode, emqx_ctl, eval_erl, [Parsed]) of
+            case rpc:call(TargetNode, emqx_ctl, run_command, [eval_erl, Parsed], infinity) of
                 {ok, Value} ->
-                    io:format("~p~n",[Value]);
+                    io:format("~p~n", [Value]);
                 {badrpc, Reason} ->
                     io:format("RPC to ~p failed: ~p~n", [TargetNode, Reason]),
                     halt(1)