Fix to pass tests #17

Merged
merged 2 commits into from Apr 12, 2012

Projects

None yet

2 participants

Contributor

This fix makes the tests pass. The problem was that while get('/', handler) handles with or without that trailing '/', the route get('/*', handler) won't work without the trailing '/', so now we use all('*', handler)

ForbesLindesay added some commits Apr 12, 2012
@ForbesLindesay ForbesLindesay Join the * onto the namespace manually to support home routes which d…
…on't end in '/'


e.g. app.namespace('/user', middleware, function(){});

middleware should now be called for '/user' as well as '/user/'.
5c76ef6
@ForbesLindesay ForbesLindesay Fix stack overflow exception caused by re-use of namespace within pro…
…xying methods.
bc6108f
@tj tj merged commit d2bc89e into expressjs:master Apr 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment