Web App will always now print automatically command result, because f…

…ew commands missed confirmation and were confusing users
openshwprojects · Feb 18, 2023 · 30f818a · 30f818a
1 parent 5107961
commit 30f818a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/cmnds/cmd_public.h b/src/cmnds/cmd_public.h
@@ -225,6 +225,8 @@ void CMD_StartTCPCommandLine();
 // cmd_script.c
 int CMD_GetCountActiveScriptThreads();
 
+const char *CMD_GetResultString(commandResult_t r);
+
 void SVM_RunThreads(int deltaMS);
 void CMD_InitScripting();
 byte* LFS_ReadFile(const char* fname);

diff --git a/src/httpserver/rest_interface.c b/src/httpserver/rest_interface.c
@@ -1563,8 +1563,12 @@ static int http_rest_post_channels(http_request_t* request) {
 
 
 static int http_rest_post_cmd(http_request_t* request) {
-	char* cmd = request->bodystart;
-	CMD_ExecuteCommand(cmd, COMMAND_FLAG_SOURCE_CONSOLE);
+	commandResult_t res;
+	const char* cmd = request->bodystart;
+	res = CMD_ExecuteCommand(cmd, COMMAND_FLAG_SOURCE_CONSOLE);
+	if (1) {
+		addLogAdv(LOG_INFO, LOG_FEATURE_CMD, "[WebApp Cmd '%s' Result] %s", cmd, CMD_GetResultString(res));
+	}
 	return http_rest_error(request, 200, "OK");
 }