fix: k8s escape resource filename on windows os

Signed-off-by: chenk <[email protected]>

Author: chen-keinan

pkg/k8s/scanner/io.go

@@ -3,6 +3,8 @@ package scanner
 import (
 	"fmt"
 	"os"
+	"regexp"
+	"runtime"
 
 	"golang.org/x/xerrors"
 	"gopkg.in/yaml.v3"
@@ -15,6 +17,10 @@ import (
 func createTempFile(artifact *artifacts.Artifact) (string, error) {
 	filename := fmt.Sprintf("%s-%s-%s-*.yaml", artifact.Namespace, artifact.Kind, artifact.Name)
 
+	if runtime.GOOS == "windows" {
+		//removes characters not permitted in file/directory names on Windows
+		filename = filenameWindowsFriendly(filename)
+	}
 	file, err := os.CreateTemp("", filename)
 	if err != nil {
 		return "", xerrors.Errorf("creating tmp file error: %w", err)
@@ -38,3 +44,9 @@ func removeFile(filename string) {
 		log.Logger.Errorf("failed to remove temp file %s: %s:", filename, err)
 	}
 }
+
+var r, _ = regexp.Compile("\\\\|/|:|\\*|\\?|<|>")
+
+func filenameWindowsFriendly(name string) string {
+	return r.ReplaceAllString(name, "_")
+}

pkg/k8s/scanner/io_test.go

@@ -0,0 +1,34 @@
+package scanner
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func Test_FilenameWindowsFriendly(t *testing.T) {
+
+	tests := []struct {
+		name     string
+		fileName string
+		want     string
+	}{
+		{
+			name:     "name with invalid char - colon",
+			fileName: `kube-system-Role-system:controller:bootstrap-signer-2934213283.yaml`,
+			want:     `kube-system-Role-system_controller_bootstrap-signer-2934213283.yaml`,
+		},
+		{
+			name:     "name with no invalid chars",
+			fileName: `kube-system-Role-system-controller-bootstrap-signer-2934213283.yaml`,
+			want:     `kube-system-Role-system-controller-bootstrap-signer-2934213283.yaml`,
+		},
+	}
+
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			got := filenameWindowsFriendly(test.fileName)
+			assert.Equal(t, test.want, got)
+		})
+	}
+}