Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http url matching can support prefix matching or regular matching #218

Open
macbobo opened this issue Dec 25, 2024 · 3 comments
Open

http url matching can support prefix matching or regular matching #218

macbobo opened this issue Dec 25, 2024 · 3 comments
Assignees

Comments

@macbobo
Copy link

macbobo commented Dec 25, 2024

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Title: http url matching can support prefix matching or regular matching

@spencercjh
Copy link
Contributor

spencercjh commented Dec 25, 2024

if filter.TargetPath != "" && filter.TargetPath != req.Path {
return false
}
if len(filter.TargetMethods) > 0 && !slices.Contains(filter.TargetMethods, req.Method) {
return false
}
if filter.TargetHostName != "" && filter.TargetHostName != req.Host {
return false

We need to enhance this part of the user namespace code and I can do this. ✌️

@hengyoush
Copy link
Owner

if filter.TargetPath != "" && filter.TargetPath != req.Path {
return false
}
if len(filter.TargetMethods) > 0 && !slices.Contains(filter.TargetMethods, req.Method) {
return false
}
if filter.TargetHostName != "" && filter.TargetHostName != req.Host {
return false

We need to enhance this part of the user namespace code and I can do this. ✌️

Go ahead!

@hengyoush hengyoush changed the title http url匹配可以支持前缀匹配或者正则匹配 http url matching can support prefix matching or regular matching Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants