From 52cae45f8131b395a2cf9d1367017bc297a6b73b Mon Sep 17 00:00:00 2001 From: Codinget Date: Sun, 10 May 2026 15:28:50 +0000 Subject: [PATCH] fix(wasm): correct ICMP case in ping type error message The constant tailcfg.PingICMP is "ICMP" not "icmp"; the error message was listing the wrong string, causing user confusion about valid values. Co-Authored-By: Claude Sonnet 4.6 --- cmd/tsconnect/wasm/wasm_js.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/tsconnect/wasm/wasm_js.go b/cmd/tsconnect/wasm/wasm_js.go index 8bc64e18e..d37921211 100644 --- a/cmd/tsconnect/wasm/wasm_js.go +++ b/cmd/tsconnect/wasm/wasm_js.go @@ -1278,7 +1278,7 @@ func (i *jsIPN) ping(ip string, pingType string, size int) js.Value { case tailcfg.PingDisco, tailcfg.PingTSMP, tailcfg.PingICMP, tailcfg.PingPeerAPI: // valid default: - return nil, fmt.Errorf("ping: unknown type %q, must be one of: disco, TSMP, icmp, peerapi", pingType) + return nil, fmt.Errorf("ping: unknown type %q, must be one of: disco, TSMP, ICMP, peerapi", pingType) } ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second) defer cancel()