fix(filters): fix suffix filters and perf optimizations

Author: samber

filters.go

@@ -19,30 +19,22 @@ func Ignore(filter Filter) Filter {
 
 // Method
 func AcceptMethod(methods ...string) Filter {
-	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
-		reqMethod := strings.ToLower(r.Method)
-
-		for _, method := range methods {
-			if strings.ToLower(method) == reqMethod {
-				return true
-			}
-		}
+	for i := range methods {
+		methods[i] = strings.ToLower(methods[i])
+	}
 
-		return false
+	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
+		return slices.Contains(methods, strings.ToLower(r.Method))
 	}
 }
 
 func IgnoreMethod(methods ...string) Filter {
-	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
-		reqMethod := strings.ToLower(r.Method)
-
-		for _, method := range methods {
-			if strings.ToLower(method) == reqMethod {
-				return false
-			}
-		}
+	for i := range methods {
+		methods[i] = strings.ToLower(methods[i])
+	}
 
-		return true
+	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
+		return !slices.Contains(methods, strings.ToLower(r.Method))
 	}
 }
 
@@ -160,10 +152,10 @@ func IgnorePathPrefix(prefixs ...string) Filter {
 	}
 }
 
-func AcceptPathSuffix(prefixs ...string) Filter {
+func AcceptPathSuffix(suffixs ...string) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
-		for _, prefix := range prefixs {
-			if strings.HasPrefix(r.URL.Path, prefix) {
+		for _, suffix := range suffixs {
+			if strings.HasSuffix(r.URL.Path, suffix) {
 				return true
 			}
 		}
@@ -187,7 +179,7 @@ func IgnorePathSuffix(suffixs ...string) Filter {
 func AcceptPathMatch(regs ...regexp.Regexp) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
 		for _, reg := range regs {
-			if reg.Match([]byte(r.URL.Path)) {
+			if reg.MatchString(r.URL.Path) {
 				return true
 			}
 		}
@@ -199,7 +191,7 @@ func AcceptPathMatch(regs ...regexp.Regexp) Filter {
 func IgnorePathMatch(regs ...regexp.Regexp) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
 		for _, reg := range regs {
-			if reg.Match([]byte(r.URL.Path)) {
+			if reg.MatchString(r.URL.Path) {
 				return false
 			}
 		}
@@ -269,10 +261,10 @@ func IgnoreHostPrefix(prefixs ...string) Filter {
 	}
 }
 
-func AcceptHostSuffix(prefixs ...string) Filter {
+func AcceptHostSuffix(suffixs ...string) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
-		for _, prefix := range prefixs {
-			if strings.HasPrefix(r.URL.Host, prefix) {
+		for _, suffix := range suffixs {
+			if strings.HasSuffix(r.URL.Host, suffix) {
 				return true
 			}
 		}
@@ -296,7 +288,7 @@ func IgnoreHostSuffix(suffixs ...string) Filter {
 func AcceptHostMatch(regs ...regexp.Regexp) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
 		for _, reg := range regs {
-			if reg.Match([]byte(r.URL.Host)) {
+			if reg.MatchString(r.URL.Host) {
 				return true
 			}
 		}
@@ -308,7 +300,7 @@ func AcceptHostMatch(regs ...regexp.Regexp) Filter {
 func IgnoreHostMatch(regs ...regexp.Regexp) Filter {
 	return func(ww middleware.WrapResponseWriter, r *http.Request) bool {
 		for _, reg := range regs {
-			if reg.Match([]byte(r.URL.Host)) {
+			if reg.MatchString(r.URL.Host) {
 				return false
 			}
 		}