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.

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

Fetch body streams are not full duplex #990

Closed
ronag opened this issue Aug 20, 2021 · 1 comment
Closed

Fetch body streams are not full duplex #990

ronag opened this issue Aug 20, 2021 · 1 comment
Labels
Docs Changes related to the documentation fetch good first issue Good for newcomers spec

Comments

@ronag
Copy link
Member

ronag commented Aug 20, 2021

The fetch spec doesn't seem to allow reading the response before the full request body has been transmitted. We are in violation with the spec here since we do allow this. Also we allow aborting the request body if the full response body has been read.

I think we should probably keep our current behavior. There doesn't seem any specific reason to why the fetch spec wouldn't allow it other than lack of resources. Just needs to add a section in docs describing it.

Refs: whatwg/fetch#1254

@ronag ronag added Docs Changes related to the documentation fetch good first issue Good for newcomers spec labels Aug 20, 2021
@szmarczak
Copy link
Member

I think it's full-duplex? whatwg/fetch@3a41b6f web-platform-tests/wpt#7693

@ronag ronag closed this as completed Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Changes related to the documentation fetch good first issue Good for newcomers spec
Projects
None yet
Development

No branches or pull requests

2 participants