Stream.pipeline does not re-emit errors without callback #20303

Activity

1. addaleax

   added

   streamIssues and PRs related to Node.js streams.Issues and PRs related to Node.js streams.

   on Apr 25, 2018
2. addaleax

   changed the title [-]Stream.pipeline shipped with a huge bug?[/-] [+]Stream.pipeline does not re-emit errors without callback[/+] on Apr 25, 2018
3. addaleax commented on Apr 25, 2018

   

   addaleax

   on Apr 25, 2018

   Member

   More actions

   You already seem to have a PR for this, would you be willing to port that?
4. phated commented on Apr 25, 2018

   

   phated

   on Apr 25, 2018

   ContributorAuthor

   More actions

   Yeah, I can try. I might have some time later today.
5. phated commented on Apr 26, 2018

   

   phated

   on Apr 26, 2018

   ContributorAuthor

   More actions

   @addaleax working on a PR now but it seems that I can't run the test suite without xcode 😞
6. addaleax commented on Apr 27, 2018

   

   addaleax

   on Apr 27, 2018

   Member

   More actions

   @phated If it would be easier for you if somebody else took this over at some point, I think that wouldn’t be an issue either
7. phated

   added a commit that references this issue on May 30, 2018

   stream: ensure Stream.pipeline re-throws errors without callback

   ...

   44d3fe3
8. addaleax

   closed this as completedin c1012b4on May 31, 2018
9. addaleax

   added a commit that references this issue on May 31, 2018

   stream: ensure Stream.pipeline re-throws errors without callback

   ...

   d1e81b0
10. sjarmak

    added a commit that references this issue on Jul 27, 2026

    stream: ensure Stream.pipeline re-throws errors without callback

    ...

    c1012b4