From 0674d9f4e470220062fefdb0200073059f7b3601 Mon Sep 17 00:00:00 2001 From: Terry Howe Date: Tue, 1 Sep 2026 15:08:19 -0600 Subject: [PATCH] refactor(registry): drop host-path login warning superseded by namespaced login warnIfHostHasPath warned that registry login only supports a hostname and not a repository path. Namespaced login now supports exactly that, so the warning is no longer accurate and the helper has no call site. Signed-off-by: Terry Howe --- pkg/registry/client.go | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/pkg/registry/client.go b/pkg/registry/client.go index 11dba0162..3954712e2 100644 --- a/pkg/registry/client.go +++ b/pkg/registry/client.go @@ -428,17 +428,6 @@ type ( } ) -// warnIfHostHasPath checks if the host contains a repository path and logs a warning if it does. -// Returns true if the host contains a path component (i.e., contains a '/'). -func warnIfHostHasPath(host string) bool { - if strings.Contains(host, "/") { - registryHost, _, _ := strings.Cut(host, "/") - slog.Warn("registry login currently only supports registry hostname, not a repository path", "host", host, "suggested", registryHost) - return true - } - return false -} - // noopStore is a credentials.Store that performs no persistence. It is used // during namespaced login to verify credentials with a registry ping without // causing remote.Login to store credentials under the hostname-only key.