support - for uri

lib/lor/lib/trie.lua

@@ -16,11 +16,11 @@ local Node = require("lor.lib.node")
 local NodeHolder = holder.NodeHolder
 local Matched = holder.Matched
 local mixin = utils.mixin
-local valid_segment_tip = "valid path should only contains: [A-Za-z0-9._-~]"
+local valid_segment_tip = "valid path should only contains: [A-Za-z0-9._%-~]"
 
 
 local function check_segment(segment)
-    local tmp = string_gsub(segment, "([A-Za-z0-9._-~]+)", "")
+    local tmp = string_gsub(segment, "([A-Za-z0-9._%-~]+)", "")
     if tmp ~= "" then
         return false
     end

spec/cases/uri_char_spec.lua

@@ -0,0 +1,45 @@
+before_each(function()
+    lor = _G.lor
+    app = lor({
+        debug = false
+    })
+    Request = _G.request
+    Response = _G.response
+    req = Request:new()
+    res = Response:new()
+
+    flag = 0
+end)
+
+after_each(function()
+    lor = nil
+    app = nil
+    Request = nil
+    Response = nil
+    req = nil
+    res = nil
+    match = nil
+end)
+
+describe("uri should support `-`", function()
+    it("test case 1", function()
+        req.path = "/a-b-c"
+        req.method = "get"
+        app:get("/a-b-c", function(req, res, next)
+            flag = 1
+        end)
+        app:handle(req, res)
+        assert.is.equals(1, flag)
+    end)
+
+    it("test case 2", function()
+        req.path = "/a_-b-/cde/-f"
+        req.method = "get"
+        app:get("/a_-b-/cde/-f", function(req, res, next)
+            flag = 2
+        end)
+        app:handle(req, res)
+        assert.is.equals(2, flag)
+    end)
+
+end)