fix: Only append slash when missing for Artifactory repoURL (#11812)

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
argoproj · Sep 13, 2023 · 4591af6 · 4591af6
1 parent cff8b83
commit 4591af6
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/pkg/apis/workflow/v1alpha1/artifact_repository_types.go b/pkg/apis/workflow/v1alpha1/artifact_repository_types.go
@@ -3,6 +3,7 @@ package v1alpha1
 import (
 	"fmt"
 	"path"
+	"strings"
 )
 
 var (
@@ -131,9 +132,9 @@ type ArtifactoryArtifactRepository struct {
 }
 
 func (r *ArtifactoryArtifactRepository) IntoArtifactLocation(l *ArtifactLocation) {
-	url := ""
-	if r.RepoURL != "" {
-		url = r.RepoURL + "/"
+	url := r.RepoURL
+	if !strings.HasSuffix(url, "/") {
+		url = url + "/"
 	}
 	k := r.KeyFormat
 	if k == "" {