bugfix: return false if not matched the fully request URI. (#52)

api7 · Aug 14, 2020 · 09d07bf · 09d07bf
1 parent 5b1d055
commit 09d07bf
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 0 deletions.
diff --git a/lib/resty/radixtree.lua b/lib/resty/radixtree.lua
@@ -452,6 +452,10 @@ local function compare_param(req_path, route, opts)
         return false
     end
 
+    if m[0] ~= req_path then
+        return false
+    end
+
     if not opts.matched then
         return true
     end

diff --git a/t/parameter.t b/t/parameter.t
@@ -235,3 +235,44 @@ match meta: metadata /name
 match meta: nil
 --- error_log
 pcre pat:
+
+
+
+=== TEST 8: /:name/foo
+--- config
+    location /t {
+        content_by_lua_block {
+            local json = require("cjson.safe")
+            local radix = require("resty.radixtree")
+            local rx = radix.new({
+                {
+                    paths = {"/:name/foo"},
+                    metadata = "metadata /:name/foo",
+                },
+            })
+
+            local opts = {matched = {}}
+            local meta = rx:match("/json/", opts)
+            ngx.say("match meta: ", meta)
+
+            meta = rx:match("/json/bar", opts)
+            ngx.say("match meta: ", meta)
+
+            meta = rx:match("/json/foo", opts)
+            ngx.say("match meta: ", meta)
+
+            meta = rx:match("/json/foo/bar", opts)
+            ngx.say("match meta: ", meta)
+        }
+    }
+--- request
+GET /t
+--- no_error_log
+[error]
+--- response_body
+match meta: nil
+match meta: nil
+match meta: metadata /:name/foo
+match meta: nil
+--- error_log
+pcre pat: