Optional parameter strategy for stream.Readable.toWeb()?

[rauschma]

What is the problem this feature will solve?

Currently, it is not possibly to customize the queueing strategy of the ReadableStream returned by stream.Readable.toWeb().

What is the feature you are proposing to solve the problem?

Change the signature from:

stream.Readable.toWeb(streamReadable: Readable): ReadableStream

stream.Readable.toWeb(streamReadable: Readable, strategy?: QueuingStrategy): ReadableStream

What alternatives have you considered?

I’m not aware of any alternatives: As far as I know, there is no way to change the queuing strategy of a ReadableStream after its constructor was invoked.

[Trott]

@nodejs/streams

[benjamingr]

@rauschma hey Axel, sure this is a pretty easy fix, the code is in /lib/internal/webstreams/adapters.js line 389, the code currently just does:

  const strategy =
    objectMode ?
      new CountQueuingStrategy({ highWaterMark }) :
      { highWaterMark };

Would you be interested in contributing a fix? (Happy to guide you through)

[Warkanlock]

I took it upon myself to make a change to include a strategy parameter in the .toWeb() method inside the Readable.js stream object.

This would be my first PR against the Node code base, please don't be too hard on me lol.

The tests are pending but I would love to hear how I can approach them.

[github-actions]

There has been no activity on this feature request for 5 months and it is unlikely to be implemented. It will be closed 6 months after the last non-automated comment.

For more information on how the project manages feature requests, please consult the feature request management document.

[benjamingr]

This already landed 🎉