Bugs with delayed-stream .maxDataSize set to Infinity

[narychen]

nodejs/node#16706 (comment)

According to the report issue, there will be problem with this. Why .maxDataSize still set to Infinity in this repo ?

[mildsunrise]

I think you mean:

node-combined-stream/lib/combined_stream.js

Line 45 in 3593923

maxDataSize: Infinity,

Reducing maxDataSize won't fix the problem, because when the limit is reached an error will be thrown.

Instead, delayed-stream should be modified to apply constant backpressure to the stream it wraps. If you have a test demonstrating nodejs/node#16706, please post it and I'll work on a fix for delayed-stream. Meanwhile, I think this issue should be closed.