ReadableStream pipeTo() should throw on error

[gmaclennan]

Describe the bug

If you use readable.pipeTo(writable) it should throw if there is an error from the destination during write. Currently any errors are caught and the writable is aborted, but the pipeTo() method resolves without an error.

To Reproduce Steps to reproduce the behavior:

Pipe the ponyfilled Readable stream to a writable stream that errors. The pipeTo() returned promise will resolve without error.

Expected behavior

pipeTo() should throw on error.

Environment:

N/A

Additional context

This is important because the ponyfilled ReadableStream implementation is used by default in @whatwg-node/server and this bug causes issues consuming request.body.

[coderabbitai]

📝 CodeRabbit Plan Mode

Generate an implementation plan and prompts that you can use with your favorite coding agent.

• Create Plan

Examples

• Example 1
• Example 2

_{You can configure auto-planning by selecting labels in the issue_enrichment configuration.}

🔗 Related PRs

#2632 - Avoid stream/promises pipeline [merged]

👤 Suggested Assignees

• ardatan

---

🧪 Issue enrichment is currently in early access.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

[ardatan]

Could you share a reproduction with a PR maybe? Thanks!