spacewander commented on a change in pull request #3987:
URL: https://github.com/apache/apisix/pull/3987#discussion_r608274953
##########
File path: docs/en/latest/router-radixtree.md
##########
@@ -71,7 +71,98 @@ Here are the rules:
|/blog/foo/gloo | `/blog/foo/*` |
|/blog/bar | not match |
-#### 4. Parameter match
+#### 4. Different routes have the same `uri`
+
+When different routes have the same uri, you can set the priority field of the
route to determine which route to match first, or add other matching rules to
distinguish different routes.
Review comment:
Need to quote the `uri`.
##########
File path: docs/en/latest/router-radixtree.md
##########
@@ -75,7 +75,7 @@ Here are the rules:
When different routes have the same uri, you can set the priority field of the
route to determine which route to match first, or add other matching rules to
distinguish different routes.
-Note: In matching rules, the `priority` field takes precedence over other
rules. Therefore, we can set other matching rules only when the value of the
`priority` field is the same.
+Note: In matching rules, the `priority` field takes precedence over other
rules.
Review comment:
We should mention that the `other rules` don't include the `uri` rule.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
