Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make invalid use of response.pipe() throw #1188

Closed
kornelski opened this issue Feb 23, 2017 · 1 comment
Closed

Make invalid use of response.pipe() throw #1188

kornelski opened this issue Feb 23, 2017 · 1 comment
Labels
Semver-Major
Milestone
4.0

Comments

@kornelski
Copy link
Contributor

It's a common pitfall to use Response.pipe() instead of Request.pipe(). We should make res.pipe() throw or even completely remove it.
This was referenced Feb 23, 2017
Closed
Closed
Closed
@skeggse
Copy link

skeggse commented Jun 20, 2017

yes, please!

@zhaoqize zhaoqize mentioned this issue Jan 18, 2018
Open
@kornelski kornelski added this to the 4.0 milestone Apr 18, 2018
kornelski added a commit that referenced this issue Jul 25, 2018
@kornelski
Block use of closed stream
b5d894f 
#1188
@kousu kousu mentioned this issue Jun 30, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Semver-Major
Projects
None yet
Milestone
4.0
Development

No branches or pull requests
2 participants
@kornelski @skeggse