Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

req.resume(); for http streams2 compat #186

Merged
merged 1 commit into from

2 participants

@rvagg

I couldn't get vows to run, I guess another 0.10 problem? So I didn't even bother looking for a relevant test, let alone write a new one for it.

@yawnt yawnt merged commit 6d55355 into flatiron:master

1 check passed

Details default The Travis build passed
@yawnt

thanks!

@rvagg

@yawnt could we get a release out of this perhaps?
Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 16, 2013
  1. @rvagg
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 0 deletions.
  1. +3 −0  lib/director/http/index.js
View
3  lib/director/http/index.js
@@ -187,6 +187,9 @@ Router.prototype.dispatch = function (req, res, callback) {
// the end event and then continue
//
req.once('end', parseAndInvoke);
+ // Streams2 requires us to start the stream if we're not explicitly
+ // reading from it.
+ req.resume();
}
else {
//
Something went wrong with that request. Please try again.