Fix router.use to accept array of middleware without path

History.md

@@ -2,6 +2,7 @@ unreleased
 ==========
 
   * Fix regression for empty string `path` in `app.use`
+  * Fix `router.use` to accept array of middleware without path
   * Improve error message for bad `app.use` arguments
 
 4.9.1 / 2014-09-16

lib/router/index.js

@@ -412,9 +412,19 @@ proto.use = function use(fn) {
   var self = this;
 
   // default path to '/'
+  // disambiguate router.use([fn])
   if (typeof fn !== 'function') {
-    offset = 1;
-    path = fn;
+    var arg = fn;
+
+    while (Array.isArray(arg) && arg.length !== 0) {
+      arg = arg[0];
+    }
+
+    // first arg is the path
+    if (typeof arg !== 'function') {
+      offset = 1;
+      path = fn;
+    }
   }
 
   var callbacks = utils.flatten(slice.call(arguments, offset));

test/Router.js

@@ -310,6 +310,28 @@ describe('Router', function(){
       router.use.bind(router, '/', null).should.throw(/requires middleware function.*Null/)
       router.use.bind(router, '/', new Date()).should.throw(/requires middleware function.*Date/)
     })
+
+    it('should accept array of middleware', function(done){
+      var count = 0;
+      var router = new Router();
+
+      function fn1(req, res, next){
+        assert.equal(++count, 1);
+        next();
+      }
+
+      function fn2(req, res, next){
+        assert.equal(++count, 2);
+        next();
+      }
+
+      router.use([fn1, fn2], function(req, res){
+        assert.equal(++count, 3);
+        done();
+      });
+
+      router.handle({ url: '/foo', method: 'GET' }, {}, function(){});
+    })
   })
 
   describe('.param', function() {