support escaping param indicator in route paths

diff --git a/lib/router/index.js b/lib/router/index.js
index a13ac47d62f083e588d047f7d556f7502f4b91e4..f0d2d945a53909c327ab1661c54e83e9a6cfe097 100644 (file)
--- a/lib/router/index.js
+++ b/lib/router/index.js
@@ -61,17 +61,37 @@ class Router {
   _pathToRoutePath(rawPath) {
     const routePath = rawPath
       .split('/')
-      .map((p) => p.startsWith(this.paramPrefix) ? new PathParameter(p.slice(this.paramPrefix.length)) : p);
+      .map((p) => this._pathPartMunge(p));
+
     if (this.ignoreTrailingSlash
     &&  routePath[routePath.length - 1] === '') {
       routePath.pop();
     }
+
     routePath[kPathMethods] = {};
     Object.freeze(routePath);
     return routePath;
   }
 
 
+  /*
+   * Convert a path part string to parameter if needed.
+   * Remove escape from an escaped parameter.
+   * @param {String} part
+   * @returns {PathParameter|String}
+   * @private
+   */
+  _pathPartMunge(part) {
+    if (part.startsWith(this.paramPrefix)) {
+      return new PathParameter(part.slice(this.paramPrefix.length));
+    }
+    if (part.startsWith('\\' + this.paramPrefix)) {
+      return part.slice(1);
+    }
+    return part;
+  }
+
+
   /**
    * Compare checkPath to fixedPath, no param substitution, params must match.
    * @param {Router~RoutePath} routePath 

diff --git a/test/lib/router.js b/test/lib/router.js
index d63917087d1490021d5441564669008c60f1676c..f9a2401eba045bb3b0972acac39287a07a907a01 100644 (file)
--- a/test/lib/router.js
+++ b/test/lib/router.js
@@ -46,6 +46,13 @@ describe('Router', function () {
       const r = router._pathToRoutePath(p);
       assert.deepStrictEqual(r, expected);
     });
+    it('defines a path with escaped parameter', function () {
+      const p = '/a/\\:b/c';
+      const expected = ['', 'a', ':b', 'c'];
+      expected[Router.kPathMethods] = {};
+      const r = router._pathToRoutePath(p);
+      assert.deepStrictEqual(r, expected);
+    });
   }); // _pathToRoutePath
 
   describe('_pathCompareExact', function () {