Skip to content

Commit cecf279

Browse files
committed
Use CSV split function instead of strings.Split
Signed-off-by: nzbr <[email protected]>
1 parent d9ce106 commit cecf279

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

pkg/requestgen/generator.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ import (
2929

3030
cmapi "github.com/cert-manager/cert-manager/pkg/apis/certmanager/v1"
3131
cmmeta "github.com/cert-manager/cert-manager/pkg/apis/meta/v1"
32+
cmutil "github.com/cert-manager/cert-manager/pkg/util"
3233
cmpki "github.com/cert-manager/cert-manager/pkg/util/pki"
3334
"github.com/cert-manager/csi-lib/manager"
3435
"github.com/cert-manager/csi-lib/metadata"
@@ -95,7 +96,10 @@ func RequestForMetadata(meta metadata.Metadata) (*manager.CertificateRequestBund
9596
if err != nil {
9697
return nil, fmt.Errorf("%q: %w", v, err)
9798
}
98-
*k = strings.Split(e, ",")
99+
*k, err = cmutil.SplitWithEscapeCSV(e)
100+
if err != nil {
101+
return nil, fmt.Errorf("%q: %w", v, err)
102+
}
99103
}
100104
}
101105
}

0 commit comments

Comments
 (0)