Updated comments for logger middleware #685

Merged
merged 2 commits into from Nov 5, 2012

Projects

None yet

2 participants

@rsolomo
Contributor
rsolomo commented Nov 4, 2012

connect.logger(function(req, res){ return 'some format string' }) actually ends up being called with the arguments (exports, req, res). I've updated the comments to match the behavior.

@rsolomo rsolomo Updated logger middleware comments
connect.logger(function(req, res){ return 'some format string' }) actually ends up being called with the arguments exports, req, and res. I've updated the comments to match the behavior.
a6fabf7
@tj
Member
tj commented Nov 5, 2012

that should be toks or tokens instead of exports, them being defined on the exports is just an implementation detail

@rsolomo
Contributor
rsolomo commented Nov 5, 2012

Okay, I've changed it to be tokens.

@tj
Member
tj commented Nov 5, 2012

awesome thanks man

@tj tj merged commit df67906 into senchalabs:master Nov 5, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment