Skip to content

Fix bikeshed errors in fetch callbacks#164

Open
jakearchibald wants to merge 3 commits intomainfrom
new-fetch-args
Open

Fix bikeshed errors in fetch callbacks#164
jakearchibald wants to merge 3 commits intomainfrom
new-fetch-args

Conversation

@jakearchibald
Copy link
Copy Markdown
Collaborator

No description provided.

@jakearchibald
Copy link
Copy Markdown
Collaborator Author

@annevk I don't have a good solution for [=body/wait=]. The intent is to wait for streaming request bodies to be consumed. Do we still need this?

Comment thread index.bs
1. [=/Fetch=] |request| with the following arguments:
<dl class="compact">
: [=fetch/useParallelQueue=]
:: True.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@annevk I know the fetch spec says I shouldn't need this, but my intent is to be able to do this processing without any page open.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tricky aspect is how we enforce any of the things that Fetch requires a client for, such as referrer and CSP. It seems you would have to obtain these ahead of time and pass them in somehow, but there's no infrastructure for that as of yet.

Comment thread index.bs
Comment on lines +299 to +303
:: The following steps given |chunkLength| (a number):
1. Increment |bgFetch|'s [=background fetch/uploaded=] by |chunkLength|.
1. [=Update background fetch instances=] for |bgFetch|.
: [=fetch/processResponse=]
:: The following steps given |response| (a [=/response=]):
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are pretty much just re-indentings of the previous algorithm.

@annevk
Copy link
Copy Markdown

annevk commented Nov 26, 2021

With current implementations, once you have seen a response the request has been transmitted. You could wait for processRequestEndOfBody as well I suppose?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants