Skip to content

Commit ec07a03

Browse files
authored
Merge pull request #1051 from utmstack/bugfix/correlation_engine/v10
Simplify RunCommand to directly stream stdout and stderr.
2 parents d28aa68 + d597571 commit ec07a03

File tree

1 file changed

+4
-23
lines changed

1 file changed

+4
-23
lines changed

correlation/utils/os.go

Lines changed: 4 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,20 @@
11
package utils
22

33
import (
4-
"io"
5-
"log"
4+
"os"
65
"os/exec"
76
)
87

98
func RunCommand(command ...string) error {
109
cmd := exec.Command(command[0], command[1:]...)
11-
output, err := cmd.StdoutPipe()
12-
if err != nil {
13-
return err
14-
}
1510

16-
err = cmd.Start()
17-
if err != nil {
18-
return err
19-
}
11+
cmd.Stderr = os.Stderr
12+
cmd.Stdout = os.Stdout
2013

21-
err = cmd.Wait()
14+
err := cmd.Run()
2215
if err != nil {
2316
return err
2417
}
2518

26-
bytesRead, err := io.ReadAll(output)
27-
if err != nil {
28-
return err
29-
}
30-
31-
err = output.Close()
32-
if err != nil {
33-
return err
34-
}
35-
36-
log.Println(string(bytesRead))
37-
3819
return nil
3920
}

0 commit comments

Comments
 (0)