fix: apply path validation for prefix pattern during router creation (#54)

Author: vm-001

.github/workflows/examples.yml

@@ -26,7 +26,9 @@ jobs:
           luaVersion: ${{ matrix.luaVersion }}
 
       - name: install LuaRocks
-        uses: leafo/gh-actions-luarocks@v4
+        uses: leafo/gh-actions-luarocks@v5
+        with:
+          luarocksVersion: "3.12.0"
 
       - name: install dependencies
         run: |

.github/workflows/test.yml

@@ -29,7 +29,9 @@ jobs:
           luaVersion: ${{ matrix.luaVersion }}
 
       - name: install LuaRocks
-        uses: leafo/gh-actions-luarocks@v4
+        uses: leafo/gh-actions-luarocks@v5
+        with:
+          luarocksVersion: "3.12.0"
 
       - name: install dependencies
         run: |
@@ -83,7 +85,9 @@ jobs:
           openrestyVersion: ${{ matrix.openrestyVersion }}
 
       - name: install LuaRocks
-        uses: leafo/gh-actions-luarocks@v4
+        uses: leafo/gh-actions-luarocks@v5
+        with:
+          luarocksVersion: "3.12.0"
 
       - name: install dependencies
         run: |

spec/router_spec.lua

@@ -56,6 +56,50 @@ describe("Router", function()
       })
       assert.is_nil(router)
       assert.equal("unable to process route(index 1): invalid methond", err)
+
+      router, err = Router.new({
+        {
+          paths = { "abc" },
+          handler = "abc",
+        },
+      })
+      assert.is_nil(router)
+      assert.equal("invalid route(index 1): path must starts with /", err)
+
+      router, err = Router.new({
+        {
+          paths = { "/a/{*b}/c" },
+          handler = "",
+        },
+      })
+      assert.is_nil(router)
+      assert.equal("invalid route(index 1): invalid prefix pattern", err)
+      router, err = Router.new({
+        {
+          paths = { "/a/{*}/c" },
+          handler = "",
+        },
+      })
+      assert.is_nil(router)
+      assert.equal("invalid route(index 1): invalid prefix pattern", err)
+      router, err = Router.new({
+        {
+          paths = { "/a/{*b}/{*c}" },
+          handler = "",
+        },
+      })
+      assert.is_nil(router)
+      assert.equal("invalid route(index 1): invalid prefix pattern", err)
+      router, err = Router.new({
+        {
+          paths = { "/a/{*}/{*}" },
+          handler = "",
+        },
+      })
+      assert.is_nil(router)
+      assert.equal("invalid route(index 1): invalid prefix pattern", err)
+
+
     end)
     it("new() with opts argument", function()
       local router, err = Router.new({}, {

src/route.lua

@@ -4,6 +4,7 @@
 
 local ipairs = ipairs
 local str_byte = string.byte
+local str_find = string.find
 local BYTE_SLASH = str_byte("/")
 
 local Route = {}
@@ -17,7 +18,11 @@ function Route.new(route)
 
   for _, path in ipairs(route.paths) do
     if str_byte(path) ~= BYTE_SLASH then
-      return nil, "path must start with /"
+      return nil, "path must starts with /"
+    end
+    local _, pattern_idx = str_find(path, "{%*[^}]*}")
+    if pattern_idx ~= nil and pattern_idx ~= #path then
+      return nil, "invalid prefix pattern"
     end
   end