Check if URL already contains remote write path before appending it

Signed-off-by: Douglas Camata <[email protected]>
stolostron · May 28, 2024 · 65d3b8c · 65d3b8c
1 parent 29695d9
commit 65d3b8c
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/operators/multiclusterobservability/controllers/placementrule/hub_info_secret.go b/operators/multiclusterobservability/controllers/placementrule/hub_info_secret.go
@@ -69,9 +69,13 @@ func generateHubInfoSecret(client client.Client, obsNamespace string,
 		}
 	}
 
-	obsApiURL := url.URL{
-		Host: obsAPIHost,
-		Path: operatorconfig.ObservatoriumAPIRemoteWritePath,
+	if !strings.HasSuffix(obsAPIHost, operatorconfig.ObservatoriumAPIRemoteWritePath) {
+		obsAPIHost += operatorconfig.ObservatoriumAPIRemoteWritePath
+	}
+
+	obsApiURL, err := url.Parse(obsAPIHost)
+	if err != nil {
+		return nil, err
 	}
 	if !obsApiURL.IsAbs() {
 		obsApiURL.Scheme = "https"