nic-chen commented on a change in pull request #3396:
URL: https://github.com/apache/apisix/pull/3396#discussion_r562624813



##########
File path: apisix/init.lua
##########
@@ -324,6 +324,28 @@ function _M.http_access_phase()
 
     core.ctx.set_vars_meta(api_ctx)
 
+    local uri = api_ctx.var.uri
+    if local_conf.apisix and local_conf.apisix.delete_uri_tail_slash then
+        if str_byte(uri, #uri) == str_byte("/") then
+            api_ctx.var.uri = str_sub(api_ctx.var.uri, 1, #uri - 1)
+            core.log.info("remove the end of uri '/', current uri: ",
+                          api_ctx.var.uri)
+        end
+    end
+
+    if router.api.has_route_not_under_apisix() or

Review comment:
       @spacewander 
   I thought about it again. There shouldn't have a switch here. If skip the 
`if` branch and continue execution, a 404 will occur because the route does not 
exist.
   
   And I think if need to configure the plugin for the internal api, it is more 
appropriate to configure a route and then configure the plugin instead of using 
the global rule. 
   
   What do you think ? Looking forward to your opinion. Thanks.
   




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to