Use CSV split function instead of strings.Split

Signed-off-by: nzbr <[email protected]>
cert-manager · Apr 2, 2024 · a29b355 · a29b355
1 parent fda8600
commit a29b355
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pkg/requestgen/generator.go b/pkg/requestgen/generator.go
@@ -29,6 +29,7 @@ import (
 
 	cmapi "github.com/cert-manager/cert-manager/pkg/apis/certmanager/v1"
 	cmmeta "github.com/cert-manager/cert-manager/pkg/apis/meta/v1"
+	cmutil "github.com/cert-manager/cert-manager/pkg/util"
 	cmpki "github.com/cert-manager/cert-manager/pkg/util/pki"
 	"github.com/cert-manager/csi-lib/manager"
 	"github.com/cert-manager/csi-lib/metadata"
@@ -95,7 +96,10 @@ func RequestForMetadata(meta metadata.Metadata) (*manager.CertificateRequestBund
 				if err != nil {
 					return nil, fmt.Errorf("%q: %w", v, err)
 				}
-				*k = strings.Split(e, ",")
+				*k, err = cmutil.SplitWithEscapeCSV(e)
+				if err != nil {
+					return nil, fmt.Errorf("%q: %w", v, err)
+				}
 			}
 		}
 	}