ForbesLindesay/express-route-tester

it don't work

Sequoia opened this issue · 2 comments

Sequoia commented

route: /admin
path: /admin/foo

  • Expected: ✅
  • Actual: ❌ 👈 incorrect

route: /admin/*
path: /admin/foo

  • Expected: ✅
  • Actual: ❌ 👈 incorrect

Etc.

rjmunro commented

So although this is called "express-route-tester" it should actually be called "path-to-regexp" tester (https://www.npmjs.com/package/path-to-regexp). Express uses "path-to-regexp", but at a very old version (0.1.7), which has different behaviour for *.

Unfortunately Express' documentation says to use this tool, which has been updated. Ideally the tool could have a switch to choose the old or new dialect - there is a PR that does this (#11) and you can use it here: https://wesleytodd.github.io/express-route-tester/

ForbesLindesay commented

This is now fixed