From 0b7c67f22ed86cdc36402a703088feb8036378d4 Mon Sep 17 00:00:00 2001 From: Philip Laine Date: Thu, 14 May 2026 15:27:42 +0200 Subject: [PATCH] Verify management url correct format (#254) Makes sure that we can parse the management url if configured by user. Fixes #78 Signed-off-by: Philip Laine --- cmd/main.go | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/cmd/main.go b/cmd/main.go index be4f470..3e9967a 100644 --- a/cmd/main.go +++ b/cmd/main.go @@ -3,14 +3,13 @@ package main import ( - // Import all Kubernetes client auth plugins (e.g. Azure, GCP, OIDC, etc.) - // to ensure that exec-entrypoint and run can make use of them. _ "k8s.io/client-go/plugin/pkg/client/auth" "crypto/tls" "errors" "flag" "fmt" + "net/url" "os" "path/filepath" "runtime" @@ -130,12 +129,16 @@ func main() { ctrl.SetLogger(zap.New(zap.UseFlagOptions(&opts))) - runtimeNamespace, err := getRuntimeNamespace(runtimeNamespace) + _, err := url.Parse(managementURL) + if err != nil { + setupLog.Error(err, "invalid management url") + os.Exit(1) + } + runtimeNamespace, err = getRuntimeNamespace(runtimeNamespace) if err != nil { setupLog.Error(err, "unable to get runtime namespace") os.Exit(1) } - if clientImage == "" { clientImage = version.ClientImage() }