From df40abc61066a8cad738bf90008313589c20dcbe Mon Sep 17 00:00:00 2001 From: Nick Khyl Date: Wed, 1 Jul 2026 15:06:51 -0700 Subject: [PATCH] ipn/ipnlocal: fix reporting of active ipnext extensions We borked this in 30a89ad3781aa99ee8b92e1ecfdab0f90034a02c and started including skipped extensions (e.g., conn25 when TAILSCALE_USE_WIP_CODE != 1) in the list of active ones. This doesn't have any impact other than on logging, though. Updates #cleanup Signed-off-by: Nick Khyl --- ipn/ipnlocal/extension_host.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ipn/ipnlocal/extension_host.go b/ipn/ipnlocal/extension_host.go index 0c4b1d933..1db183afe 100644 --- a/ipn/ipnlocal/extension_host.go +++ b/ipn/ipnlocal/extension_host.go @@ -7,7 +7,6 @@ import ( "context" "errors" "fmt" - "maps" "reflect" "slices" "strings" @@ -269,7 +268,10 @@ func (h *ExtensionHost) init() { // Report active extensions to the log. // TODO(nickkhyl): update client metrics to include the active/failed/skipped extensions. h.mu.Lock() - extensionNames := slices.Collect(maps.Keys(h.extensionsByName)) + var extensionNames []string + for _, ext := range h.activeExtensions { + extensionNames = append(extensionNames, ext.Name()) + } h.mu.Unlock() h.logf("active extensions: %v", strings.Join(extensionNames, ", "))