Update cmd/gateway/setup.go

kate-osborn · pleshakov · Kate Osborn · commit 29e6ea0374aa · 2022-09-20T15:00:51.000-06:00
Use re.MatchString instead of re.Match

Co-authored-by: Michael Pleshakov &lt;pleshakov@users.noreply.github.com&gt;
diff --git a/cmd/gateway/setup.go b/cmd/gateway/setup.go
@@ -59,7 +59,7 @@ func GatewayControllerParam(domain string) ValidatorContext {
 func validateControllerName(name string) error {
 	// Regex from: https://github.com/kubernetes-sigs/gateway-api/blob/547122f7f55ac0464685552898c560658fb40073/apis/v1alpha2/shared_types.go#L462
 	re := regexp.MustCompile(controllerNameRegex)
-	if !re.Match([]byte(name)) {
+	if !re.MatchString(name) {
 		return fmt.Errorf("invalid gateway controller name: %s; expected format is DOMAIN/PATH", name)
 	}
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ func GatewayControllerParam(domain string) ValidatorContext {`
`59`	`59`	`func validateControllerName(name string) error {`
`60`	`60`	`// Regex from: https://github.com/kubernetes-sigs/gateway-api/blob/547122f7f55ac0464685552898c560658fb40073/apis/v1alpha2/shared_types.go#L462`
`61`	`61`	`re := regexp.MustCompile(controllerNameRegex)`
`62`		`- if !re.Match([]byte(name)) {`
	`62`	`+ if !re.MatchString(name) {`
`63`	`63`	`return fmt.Errorf("invalid gateway controller name: %s; expected format is DOMAIN/PATH", name)`
`64`	`64`	`}`
`65`	`65`	`return nil`