fix: ensuring resourcequota name doesn't break DNS-1123

Signed-off-by: Dario Tranchitella <[email protected]>
projectcapsule · Nov 21, 2023 · 34977aa · 34977aa
1 parent 2465b66
commit 34977aa
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/api/v1beta2/tenant_annotations.go b/api/v1beta2/tenant_annotations.go
@@ -7,6 +7,7 @@ import (
 	"crypto/md5" //#nosec
 	"encoding/hex"
 	"fmt"
+	"strings"
 )
 
 const (
@@ -18,13 +19,13 @@ const (
 )
 
 func createAnnotation(format string, resource fmt.Stringer) (string, error) {
-	suffix := resource.String()
+	resourceStr := strings.ReplaceAll(resource.String(), "/", "_")
 
-	hash := md5.Sum([]byte(resource.String())) //#nosec
+	hash := md5.Sum([]byte(resourceStr)) //#nosec
 
 	hashed := hex.EncodeToString(hash[:])
 	capsuleHashed := format + hashed
-	capsuleAnnotation := format + suffix
+	capsuleAnnotation := format + resourceStr
 
 	switch {
 	case len(capsuleAnnotation) <= maxAnnotationLength: