fix bug when the route is a Null string

[ShangguanQuail]

I got an error message when I use like this:

// With express-promise-router
var router = require('express-promise-router')();

router.use('', function (req, res) {
    return Promise.reject();
})

this is the error message:

/express-promise-router/lib/express-promise-router.js:81
                ((Array.isArray(args[0]) && 'string' === typeof args[0][0]) || args[0][0] instanceof RegExp)
                                                                                      ^

TypeError: Cannot read property '0' of undefined

Yeah, the original url is a Null string, though it‘s not standard.

[mormahr]

I'm pretty busy right now. I'll probably look into this around Christmas.

Thanks for your contribution!

[mormahr]

Hi there!

Sorry for keeping you waiting! I' tried to reproduce you're problem with the current release version but wasn't able to. From the line number it seems like you're using an outdated version (1.1.1?). I checked out v1.1.1 and tried to reproduce it without any luck.

var app = require('express')();
var router = require('./lib/express-promise-router')();

router.use('', function (req, res) {
    return Promise.reject();
})

app.use(router);
app.listen(1204)

I used this sample code and it correctly set the route for '/' even though only '' was given. I used express@4.16.2 and for this and express-promise-router@1.1.1 as well as express-promise-router@2.0.0. Maybe you can provide a full example repository that reproduces your problem.

Thank you for your patience!
Moritz

[mormahr]

As this has been inactive for a few month i'm closing it. Feel free to comment if something has changed and / or your problem persists.