mirror of
https://github.com/alibaba/higress.git
synced 2026-06-09 12:47:28 +08:00
optimize rewrite regex (#801)
This commit is contained in:
@@ -83,7 +83,7 @@ func (i *Ingress) NeedRegexMatch(path string) bool {
|
|||||||
if i.Rewrite == nil {
|
if i.Rewrite == nil {
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
if strings.ContainsAny(path, `\.+*?()|[]{}^$`) {
|
if i.Rewrite.RewriteTarget != "" && strings.ContainsAny(path, `\.+*?()|[]{}^$`) {
|
||||||
return true
|
return true
|
||||||
}
|
}
|
||||||
if strings.ContainsAny(i.Rewrite.RewriteTarget, `$\`) {
|
if strings.ContainsAny(i.Rewrite.RewriteTarget, `$\`) {
|
||||||
|
|||||||
@@ -67,6 +67,15 @@ func TestNeedRegexMatch(t *testing.T) {
|
|||||||
inputPath: "/.*",
|
inputPath: "/.*",
|
||||||
expect: true,
|
expect: true,
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
input: &Ingress{
|
||||||
|
Rewrite: &RewriteConfig{
|
||||||
|
UseRegex: false,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
inputPath: "/.",
|
||||||
|
expect: false,
|
||||||
|
},
|
||||||
{
|
{
|
||||||
input: &Ingress{
|
input: &Ingress{
|
||||||
Rewrite: &RewriteConfig{
|
Rewrite: &RewriteConfig{
|
||||||
|
|||||||
Reference in New Issue
Block a user