4.0.1: req.query bug fix and new test

lib/routing/sequential.js

@@ -84,8 +84,9 @@ module.exports = (config = {}) => {
       req.path = split[0]
 
       const queryparse = req.app.__settings.get('query parser fn')
-      req.search = '?' + split[1]
-      req.query = queryparse(split[1])
+      const query = split.slice(1).join('?')
+      req.search = '?' + query
+      req.query = queryparse(query)
 
       res.locals = Object.create(null) // res.locals is rather convenient at times, so we leave it on even for API routes
       req.paramsCalled = {} // needed for app.param

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fyrejet",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "description": "Web Framework for node.js that strives to provide (almost) perfect compatibility with Express, while providing better performance, where you need it.",
   "main": "index.js",
   "scripts": {

test/req.query.js

@@ -12,6 +12,13 @@ describe('req', function () {
         .expect(200, '{}', done)
     })
 
+    it('should handle another "?" in the query', function (done) {
+      var app = createApp()
+      request(app)
+        .get('/?dogs=true?cats=true')
+        .expect(200, '{"dogs":"true?cats=true"}', done)
+    })
+
     it('should default to parse complex keys', function (done) {
       var app = createApp()