From 6893723ccfbc0c021af6290df6bd5a8ef1f35524 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 f1bd2315e..d9a5920e7 100644 --- a/cmd/tsconnect/wasm/wasm_js.go +++ b/cmd/tsconnect/wasm/wasm_js.go @@ -1284,7 +1284,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()