update regex

Author: haywoodsh

internal/k8s/validation.go

@@ -873,7 +873,7 @@ func validateBackend(backend *networking.IngressBackend, fieldPath *field.Path)
 }
 
 const (
-	pathFmt    = `/[^\s;\\]*`
+	pathFmt    = `/[^\s;]*`
 	pathErrMsg = "must start with / and must not include any whitespace character or `;`"
 )
 
@@ -950,7 +950,7 @@ func ValidateEscapedString(body string, examples ...string) error {
 }
 
 const (
-	illegalKeywordFmt    = `/etc|/root`
+	illegalKeywordFmt    = `/etc|/root|\\n|\\r`
 	illegalKeywordErrMsg = `must not contain invalid paths`
 )
 

internal/k8s/validation_test.go

@@ -3343,20 +3343,20 @@ func TestValidateRegexPath(t *testing.T) {
 		msg       string
 	}{
 		{
-			regexPath: "~ ^/foo.*\\.jpg",
+			regexPath: "/foo.*\\.jpg",
 			msg:       "case sensitive regexp",
 		},
 		{
-			regexPath: "~* ^/Bar.*\\.jpg",
+			regexPath: "/Bar.*\\.jpg",
 			msg:       "case insensitive regexp",
 		},
 		{
-			regexPath: `~ ^/f\"oo.*\\.jpg`,
+			regexPath: `/f\"oo.*\\.jpg`,
 			msg:       "regexp with escaped double quotes",
 		},
 		{
-			regexPath: "~ [0-9a-z]{4}[0-9]+",
-			msg:       "regexp with brackets",
+			regexPath: "/[0-9a-z]{4}[0-9]+",
+			msg:       "regexp with curly braces",
 		},
 	}
 
@@ -3375,19 +3375,19 @@ func TestValidateRegexPathFails(t *testing.T) {
 		msg       string
 	}{
 		{
-			regexPath: "~ [{",
+			regexPath: "[{",
 			msg:       "invalid regexp",
 		},
 		{
-			regexPath: `~ /foo"`,
+			regexPath: `/foo"`,
 			msg:       "unescaped double quotes",
 		},
 		{
-			regexPath: `~"`,
+			regexPath: `"`,
 			msg:       "empty regex",
 		},
 		{
-			regexPath: `~ /foo\`,
+			regexPath: `/foo\`,
 			msg:       "ending in backslash",
 		},
 	}
@@ -3409,6 +3409,10 @@ func TestValidatePath(t *testing.T) {
 		"/a-1/_A/",
 		"/[A-Za-z]{6}/[a-z]{1,2}",
 		"/[0-9a-z]{4}[0-9]",
+		"/foo.*\\.jpg",
+		"/Bar.*\\.jpg",
+		`/f\"oo.*\\.jpg`,
+		"/[0-9a-z]{4}[0-9]+",
 	}
 
 	for _, path := range validPaths {