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

Response.prototype.clone() should take into account the internal response. #148

Closed
yutakahirano opened this issue Oct 29, 2015 · 3 comments
Closed

Response.prototype.clone() should take into account the internal response. #148

yutakahirano opened this issue Oct 29, 2015 · 3 comments
Assignees
@yutakahirano

Comments

@yutakahirano
Copy link
Member

Cloning the body in the internal response is needed.
@yutakahirano yutakahirano self-assigned this Oct 29, 2015
@tyoshino
Copy link
Member

Note from the f2f: The result of clone() should also preserve most of the properties of the original Response. I.e. the results of cloning an opaque one should be opaque.

@annevk
Copy link
Member

annevk commented Jan 7, 2016

This is solved in https://github.com/yutakahirano/fetch-with-streams just not merged upstream yet.

@annevk
Copy link
Member

annevk commented Feb 15, 2016

This was fixed in #200 by @yutakahirano. Yay!

@annevk annevk closed this as completed Feb 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yutakahirano yutakahirano

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@annevk @tyoshino @yutakahirano