From bf7e66e045b6851be968cf13df0bed94b2254f73 Mon Sep 17 00:00:00 2001 From: rcourtman <8825017+rcourtman@users.noreply.github.com> Date: Fri, 23 Oct 2026 13:18:20 +0100 Subject: [PATCH] Drop installer NUL cases main does not implement The ingress test came from a line whose installer also refused NUL bytes in token files. Main's installer does not, so those cases failed on every runner. The remaining cases cover what this branch fixes: both token options read through the real installer and token reader. Contract-Neutral: test scope correction for the portability fix; no contract change --- cmd/pulse-agent/collector_token_ingress_unix_test.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/cmd/pulse-agent/collector_token_ingress_unix_test.go b/cmd/pulse-agent/collector_token_ingress_unix_test.go index dd57bbd787..60713aeefa 100644 --- a/cmd/pulse-agent/collector_token_ingress_unix_test.go +++ b/cmd/pulse-agent/collector_token_ingress_unix_test.go @@ -61,9 +61,6 @@ func TestCollectorTokenReaderInstallerIngress(t *testing.T) { "bare": "synthetic-secret", "LF": "synthetic-secret\n", "normalized": " \tsynthetic-secret\r\n", "literal": "PULSE_TOKEN='synthetic-secret'", "UTF8": "synthetic-caf\u00e9", "limit": strings.Repeat("s", 4096), - "NUL leading": "\x00synthetic-secret", "NUL interior": "synthetic\x00-secret", - "NUL trailing": "synthetic-secret\x00", "NUL before LF": "synthetic-secret\x00\n", - "NUL after LF": "synthetic-secret\n\x00", "NUL only": "\x00", } { for _, option := range []string{"collector", "action"} { t.Run(name+"/"+option, func(t *testing.T) {