Merge pull request #23 from basecamp/dont-panic-if-unable-to-launch-p…

…rocess Failure to start wrapped process should not panic
basecamp · May 1, 2024 · 2cfbe2c · 2cfbe2c
2 parents 550c8cd + 42ad86a
commit 2cfbe2c
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/internal/service.go b/internal/service.go
@@ -2,6 +2,7 @@ package internal
 
 import (
 	"fmt"
+	"log/slog"
 	"net/url"
 	"os"
 )
@@ -37,7 +38,8 @@ func (s *Service) Run() int {
 
 	exitCode, err := upstream.Run()
 	if err != nil {
-		panic(err)
+		slog.Error("Failed to start wrapped process", "command", s.config.UpstreamCommand, "args", s.config.UpstreamArgs, "error", err)
+		return 1
 	}
 
 	return exitCode