It will be very nice to see the tests. Thank you.

Tested. If you cherry pick that test (a32692d) and run it against the current code it should fail. I changed the test to sending 10 requests and removed res.resume() which isn't documented in the README (with res.resume this issue doesn't happen however this patch renders res.resume unnecessary).

I'm not sure whether this or adding res.resume() to the docs is a better solution but I'll leave that up to you.

Thanx for the test.
Btw, does it fail against any particular node version or all 0.6–0.11 are affected?

I'm running v0.10.15. I haven't tested it against any other node versions.

I see. I'll play with it.

So if I run your test within original codebase, it doesn't fail for any version (0.6 – 0.11), although it would take it a while to finish in 0.10 and 0.11 (Tried 0.10.12, 0.10.16 and 0.11.5).

This fix, would trigger all the requests into "old mode", which might be not desirable default behavior in 0.10. And in 0.12 it will be non-issue (Streams3 FTW). :)

As for documentation it was discussed in lengths and described here: http://nodejs.org/api/stream.html#stream_compatibility_with_older_node_versions

But you're right, we need to make it more obvious.
I updated readme and I included your (reduced) test into test suite (as separate new entity, we need old tests to make sure we don't break existing stuff).

Thanx a lot.