search

iron-meteor / iron-middleware-stack

Client and server middleware support inspired by Connect.
MIT License
8 stars 13 forks source link

Bugfix in MiddlewareStack.concat #5

Closed benmgreene closed 9 years ago

benmgreene commented 9 years ago

Hi there,

I've been stepping through the entirety of the IR process trying to figure out, um, stuffs. Doing so, I noticed a peculiarity -- when MiddlewareStacks are concatenated, there is a "length" member variable that doesn't get updated. I'm assuming this is a bug and providing a PR.

cmather commented 9 years ago

Thanks Ben!

On Jan 29, 2015, at 12:21 PM, Ben M Greene notifications@github.com wrote:

Hi there,

I've been stepping through the entirety of the IR process trying to figure out, um, stuffs. Doing so, I noticed a peculiarity -- when MiddlewareStacks are concatenated, there is a "length" member variable that doesn't get updated. I'm assuming this is a bug and providing a PR.

You can view, comment on, or merge this pull request online at:

https://github.com/EventedMind/iron-middleware-stack/pull/5

Commit Summary

Fix bug in MiddlewareStack.concat File Changes

M lib/middleware_stack.js (2) Patch Links:

https://github.com/EventedMind/iron-middleware-stack/pull/5.patch https://github.com/EventedMind/iron-middleware-stack/pull/5.diff — Reply to this email directly or view it on GitHub.