backpressure is broken about node-spdyproxy HOT 13 CLOSED

@chemhack which version of node? I'm guessing you're running the latest node-spdyproxy (from master)...?

from node-spdyproxy.

node v0.8.23

yes, node-spdyproxy from master

from node-spdyproxy.

Can you try 0.10+? We've had similar reports in the past which seems to have gone away after upgrade - see #21.

from node-spdyproxy.

Same result

from node-spdyproxy.

Update: This issue was observed in HTTP and HTTPS download.

from node-spdyproxy.

Update again:

It has already been fixed 6 days ago by @indutny

spdy-http2/node-spdy@be33b5a

After back porting this commit to 1.5.0, the backpressure is working.

@igrigorik Why not update to newer version of node-spdy? Do you have a plan for this?

@indutny Can you release a 1.5.1 version of node-spdy on npm with commit spdy-http2/node-spdy@be33b5a patched as a temporary fix?

from node-spdyproxy.

I'm happy to bump the version..

from node-spdyproxy.

Oh, that's nice :) Didn't realize that this could be a problem for
spdyproxy ;)

Cheers,
Fedor.

On Mon, May 20, 2013 at 6:38 AM, Ilya Grigorik [email protected]:

I'm happy to bump the version..

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/28#issuecomment-18129823
.

from node-spdyproxy.

@igrigorik How is going? I still didn't see version 1.5.1 of node-spdy. And node-spdyproxy has compatible issue with 1.8.x

from node-spdyproxy.

@chemhack node-spdy v1.8.8 is up on NPM, so in theory as long as you install that, node-spdyproxy should work just fine.. I'm not sure where the 1.5.x requirement came from?

from node-spdyproxy.

@igrigorik The problem is node-spdyproxy doesn't work with node-spdy 1.8.8. 1.5.0 just worked fine.

from node-spdyproxy.

Oh, well then.. That's the actual problem we need to fix, instead of releasing 1.5.1.

from node-spdyproxy.

Ok, testing locally, I believe spdy-http2/node-spdy#95 should fix the problem. /cc @indutny

from node-spdyproxy.