Merge pull request #44 from snorwin/sanitize-pid

strip non ascii from pids

Author: snorwin

main.go

@@ -8,6 +8,7 @@ import (
 	"strings"
 	"sync"
 	"syscall"
+	"unicode"
 
 	"github.com/fsnotify/fsnotify"
 	"github.com/snorwin/haproxy-reload-wrapper/pkg/exec"
@@ -183,14 +184,27 @@ func runInstance() {
 	cmds = append(cmds, cmd)
 }
 
+// pids returns the PID list as a space-separated string
 func pids() string {
 	if len(cmds) == 0 {
 		return ""
 	}
 
 	out := make([]string, 0, len(cmds))
 	for _, c := range cmds {
-		out = append(out, strconv.Itoa(c.Process.Pid))
+		out = append(out, stripCtrlChars(strconv.Itoa(c.Process.Pid)))
 	}
 	return strings.Join(out, " ")
 }
+
+// stripCtrlChars returns a sanitized string
+// the PID read from the HAProxy socket includes a trailing control character,
+// usually the multi‑byte separator that HAProxy puts at the end of the Runtime API response
+func stripCtrlChars(s string) string {
+	return strings.Map(func(r rune) rune {
+		if unicode.IsControl(r) {
+			return -1
+		}
+		return r
+	}, s)
+}