fix route lookup if have query params

MANUAL.md

@@ -289,6 +289,7 @@ See buildParseQuery.
 Change Log
 ==========
 
+- 0.6.2 - fix router path lookups if have query params
 - 0.6.1 - let createHandler also accept onRequest and onError handlers directly, not just via options
 - 0.6.0 - Breaking: remove the broken `encoding` handling, make the app decode.  Now returns Buffers.
           Fix null response handling (`null` is an object not an empty string, JSON serialize it)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "microrest",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "tiny embeddable REST web framework",
   "keywords": ["micro", "nano", "framework", "middleware", "REST", "embeddable", "fast"],
   "main": "rest.js",

rest.js

@@ -138,6 +138,8 @@ Rest.NanoRouter.prototype.setRoute = function setRoute( path, method, mwStep ) {
     this.routes[path] = mwStep;
 }
 Rest.NanoRouter.prototype.getRoute = function getRoute( path, method ) {
+    var mark = path.indexOf('?');
+    if (mark >= 0) path = path.slice(0, mark);
     var mwSteps = this.routes[path];
     while (!mwSteps && this.matchPrefix && path.length > 1) mwSteps = this.routes[path = path.slice(path, path.lastIndexOf('/')) || '/'];
     return mwSteps;

router.js

@@ -65,9 +65,10 @@ Router.prototype.deleteRoute = function deleteRoute( path, method ) {
 }
 
 Router.prototype.getRoute = function getRoute( path, method, route ) {
-    var mw;
+    var mw, mark;
 
     if (!path) return null;                             // path is required
+    if ((mark = path.indexOf('?')) >= 0) path = path.slice(0, mark);
     if (this.steps[path]) return this.steps[path];      // use-, pre-, err- and post- steps
     mw = this.maproutes[path] && (this.maproutes[path][method] || this.maproutes[path]['_ANY_']);
     if (mw) return mw;                                  // direct-mapped routes

test-rest.js

@@ -725,7 +725,7 @@ res.on('data', function(chunk) { console.log("AR: chunk", String(chunk)) });
             var router = new Rest.NanoRouter();
             router.setRoute('/path2', noop);
             t.equal(router.getRoute('/path0'), null);
-            t.deepEqual(router.getRoute('/path2'), noop);
+            t.deepEqual(router.getRoute('/path2?a=1'), noop);
             t.done();
         },
 

test-router.js

@@ -56,6 +56,12 @@ module.exports = {
             t.done();
         },
 
+        'should trim query params': function(t) {
+            this.router.setRoute('/test/path', this.fn1);
+            t.deepEqual(this.router.getRoute('/test/path?a=1&b=2'), [this.fn1]);
+            t.done();
+        },
+
         'should exact match if tail is not gathered': function(t) {
             this.router.setRoute('/:name1/:name2/other', this.fn2);
             t.ok(this.router.getRoute('/test/path/other'));