timeout option is not considered if streams are attached

[ptusch]

Hello there,

I was fiddling with the timeout option. It seems like it doesn't quite work as I'd expect (any maybe you as well!)

The setup

I have a bash script that sleeps for 6 seconds (sample.sh) and a nodejs file that is supposed to timeout after 200ms.

What's happening

The nodejs file rejects as expected but not after 200ms but after 6seconds. This is weird.
I investigated a bit and the internal timeout to kill the process is reached. The process is killed as well.

The issue seems to be this line; It looks like we are waiting for the stream of the killed process. It seems unusual since it will never talk again.
Removing the selected line (and the one after it) makes the process timeout nicely. The cleanup doesn't seem to be correct.

I've tried to work around by passing the options stdout and stderr and set each to null but that didn't really help.

Attached you'll find the sample project. Please let me know if something is missing. I might try to find a solution this evening.. ;)
execa_timeout_broken.tar.gz

gmartigny earned $30.00 by resolving this issue!

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on Issuehunt to raise funds.

[IssueHuntBot]

@IssueHunt has funded $30.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[IssueHuntBot]

@sindresorhus has rewarded $27.00 to @GMartigny. See it on IssueHunt

• 💰 Total deposit: $30.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $3.00