proxy-rewrite plugin not support multi regex_uri
Description
"proxy-rewrite": { "regex_uri": [ "(^/message/message-center-mqtt-client/(.*)|^/message/internal/message-center-mqtt-client/(.*))", "/$1" ] }
see this case
{
"proxy-rewrite":{
"regex_uri":[
"^(/message/message-center-mqtt-client/|/message/internal/message-center-mqtt-client/)(.*)",
"/$1"
]
}
}
see https://apisix.apache.org/docs/apisix/plugins/proxy-rewrite/
@ipanocloud May I know the use case or pain point that you're facing? Currently, it's indeed that only one regex_uri is supported .
I think we need to make two improvements: first, the size of the regex_uri array is marked as two on the document. later, we are doing functional optimization, multiple regex_uri supported . cc @spacewander what do you think about this.
@Hazel6869 Could you provide some reasons (use cases or the other) to make these two improvements? And what do you want to see after the improvements?
@spacewander I think the first one is the optimization of the document, the status is easy to misunderstand (ab ---> ababab), the second, multiple regex_uri supported needs to be evaluated.
Let's start by understanding what problems are solved by supporting multiple regex_uri. cc @ipanocloud
Hello, We are facing the same problem while migrating Gateway from Nginx to apisix. Cause we have lots of Rewrite rule
@Hazel6869 Please take a look when you have time.
This issue has been marked as stale due to 350 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please simply write any comment. Even if closed, you can still revive the issue at any time or discuss it on the [email protected] list. Thank you for your contributions.
This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revive the issue at any time.