Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

[fix] Args fix #87

Merged
merged 1 commit into from

2 participants

@ghost

After commt flatiron/union@81fbc95 default error handler in Union doesn't use thisArg{req,res}, so they have to be passed as callback's arguments.

@pksunkara

Thanks, I will review it.

@pksunkara pksunkara merged commit 84e283c into flatiron:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 27, 2012
  1. @dmlukichev
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/director/http/index.js
View
2  lib/director/http/index.js
@@ -97,7 +97,7 @@ Router.prototype.dispatch = function (req, res, callback) {
if (!fns || fns.length === 0) {
error = new exports.NotFound('Could not find path: ' + req.url)
if (callback) {
- callback.call(thisArg, error);
+ callback.call(thisArg, error, req, res);
}
return false;
}
Something went wrong with that request. Please try again.