Merge pull request #1053 from utmstack/bugfix/correlation_engine/v10

osmontero · web-flow · commit bbc313ed979c · 2025-02-26T14:59:19.000+02:00
Refactor `RunCommand` to separate command name and arguments
diff --git a/correlation/utils/os.go b/correlation/utils/os.go
@@ -5,8 +5,8 @@ import (
 	"os/exec"
 )
 
-func RunCommand(command ...string) error {
-	cmd := exec.Command(command[0], command[1:]...)
+func RunCommand(name string, args ...string) error {
+	cmd := exec.Command(name, args...)
 
 	cmd.Stderr = os.Stderr
 	cmd.Stdout = os.Stdout

Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,8 @@ import (`
`5`	`5`	`"os/exec"`
`6`	`6`	`)`
`7`	`7`
`8`		`-func RunCommand(command ...string) error {`
`9`		`- cmd := exec.Command(command[0], command[1:]...)`
	`8`	`+func RunCommand(name string, args ...string) error {`
	`9`	`+ cmd := exec.Command(name, args...)`
`10`	`10`
`11`	`11`	`cmd.Stderr = os.Stderr`
`12`	`12`	`cmd.Stdout = os.Stdout`