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

Process 103 Early Hints response #1404

Merged
merged 12 commits into from Apr 7, 2022
Merged

Process 103 Early Hints response #1404

Changes from 6 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
2616cad
Process 103 Early Hints response
bashi Feb 17, 2022
3316f78
103 is a null body status
bashi Feb 17, 2022
f5dc828
Add an assertion for CORS
bashi Feb 21, 2022
1f21c3c
Remove nobackref
bashi Mar 1, 2022
892843f
Merge nested if block
bashi Mar 1, 2022
d9feb0a
Drop responseStartTime variable
bashi Mar 1, 2022
7ca71c9
Pass the whole response
bashi Mar 4, 2022
1c16939
Update note on running "process early hints response"
bashi Mar 4, 2022
c173ffe
Preserve timingInfo setting behavior
bashi Mar 4, 2022
79f4578
some formatting and editorial nits
annevk Mar 10, 2022
c6c8196
move assert and add usage info
annevk Apr 6, 2022
eda2b8d
nit
annevk Apr 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
64 changes: 48 additions & 16 deletions fetch.bs
View file
Open in desktop
Expand Up @@ -199,6 +199,7 @@ lt="authentication entry">authentication entries</a> (for HTTP authentication).
<dt><dfn for="fetch params" id=fetch-params-process-request-body>process request body chunk length</dfn>
(default null)
<dt><dfn for="fetch params">process request end-of-body</dfn> (default null)
<dt><dfn for="fetch params">process early hints response</dfn> (default null)
<dt><dfn for="fetch params">process response</dfn> (default null)
<dt><dfn for="fetch params">process response end-of-body</dfn> (default null)
<dt><dfn for="fetch params">process response consume body</dfn> (default null)
Expand Down Expand Up @@ -1099,7 +1100,7 @@ format of range header value can be set using <a>add a range header</a>.
<p class=XXX>Various edge cases in mapping HTTP/1's <code>status-code</code> to this concept are
worked on in <a href=https://github.com/whatwg/fetch/issues/1156>issue #1156</a>.

<p>A <dfn export>null body status</dfn> is a <a for=/>status</a> that is 101, 204, 205, or 304.
<p>A <dfn export>null body status</dfn> is a <a for=/>status</a> that is 101, 103, 204, 205, or 304.

<p>An <dfn export>ok status</dfn> is a <a for=/>status</a> in the range 200 to 299, inclusive.

Expand Down Expand Up @@ -3744,15 +3745,19 @@ optional algorithm
<dfn export for=fetch id=process-request-body><var>processRequestBodyChunkLength</var></dfn>, an
optional algorithm
<dfn export for=fetch id=process-request-end-of-body oldids=process-request-end-of-file><var>processRequestEndOfBody</var></dfn>,
an optional algorithm <dfn export for=fetch id=process-early-hints-response>
<var>processEarlyHintsResponse</var></dfn>,
an optional algorithm <dfn export for=fetch id=process-response><var>processResponse</var></dfn>, an
optional algorithm <dfn export for=fetch><var>processResponseEndOfBody</var></dfn>, an optional
algorithm
<dfn export for=fetch id=process-response-end-of-body oldids=process-response-end-of-file><var>processResponseConsumeBody</var></dfn>,
and an optional boolean <dfn export for=fetch><var>useParallelQueue</var></dfn> (default false), run
the steps below. If given, <var>processRequestBodyChunkLength</var> must be an algorithm accepting
an integer representing the number of bytes transmitted. If given,
<var>processRequestEndOfBody</var> must be an algorithm accepting no arguments. If given,
<var>processResponse</var> must be an algorithm accepting a <a for=/>response</a>. If given,
<var>processRequestEndOfBody</var> must be an algorithm accepting no arguments.
If given, <var>processEarlyHintsResponse</var> must be an algorithm accepting a
<a for=request>header list</a>. If given, <var>processResponse</var> must be an
algorithm accepting a <a for=/>response</a>. If given,
<var>processResponseEndOfBody</var> must be an algorithm accepting a <a for=/>response</a>. If
given, <var>processResponseConsumeBody</var> must be an algorithm accepting a <a for=/>response</a>
and null, failure, or a <a for=/>byte sequence</a>.
Expand Down Expand Up @@ -3809,6 +3814,8 @@ the request.
<a for="fetch params">process request body chunk length</a> is
<var>processRequestBodyChunkLength</var>,
<a for="fetch params">process request end-of-body</a> is <var>processRequestEndOfBody</var>,
<a for="fetch params">process early hints response</a> is
<var>processEarlyHintsResponse</var>,
<a for="fetch params">process response</a> is <var>processResponse</var>,
<a for="fetch params">process response consume body</a> is <var>processResponseConsumeBody</var>,
<a for="fetch params">process response end-of-body</a> is <var>processResponseEndOfBody</var>,
Expand Down Expand Up @@ -5460,22 +5467,47 @@ optional boolean <var>forceNewConnection</var> (default false), run these steps:
<a for=request>body</a> cannot be recreated and that is why the buffer is needed.
</div>

<li><p>Set <var>timingInfo</var>'s
<a for="fetch timing info">final network-response start time</a> to the
<a for=/>coarsened shared current time</a> given <var>fetchParams</var>'s
<a for="fetch params">cross-origin isolated capability</a>, immediately after the user agent's
HTTP parser receives the first byte of the response (e.g., frame header bytes for HTTP/2 or
response status line for HTTP/1.x).
<li>
<p>While true:
<ol>
<li><p>Wait until all the <a for=/>headers</a> are transmitted.
yoavweiss marked this conversation as resolved.
Show resolved Hide resolved

<li><p>Wait until all the <a for=/>headers</a> are transmitted.
<li><p>If <var>timingInfo</var>'s
<a for="fetch timing info">final network-response start time</a> is 0, set
<var>timingInfo</var>'s <a for="fetch timing info">final network-response start time</a> to
<a for=/>coarsened shared current time</a> given <var>fetchParams</var>'s
<a for="fetch params">cross-origin isolated capability</a>, immediately after the user
agent's HTTP parser receives the first byte of the response (e.g., frame header bytes for
HTTP/2 or response status line for HTTP/1.x).

<li>
<p>Any <a for=/>responses</a> whose <a for=response>status</a> is in the range 100 to 199,
inclusive, and is not 101, are to be ignored, except for the purposes of setting
<var>timingInfo</var>'s <a for="fetch timing info">final network-response start time</a> above.
<li><p>Let <var>status</var> be <a for=response>status</a> of <a for=/>response</a>.

<li><p>If <var>status</var> is in the range 100 to 199, inclusive:
<ol>
<li><p>If <var>status</var> is 101, <a for=iteration>break</a>.

<li><p>If <var>status</var> is 103 and <var>fetchParams</var>'s
<a for="fetch params">process early hints response</a> is non-null, then:
<ol>
<li><p><a for=/>Assert</a>: <var>request</var>'s <a for=request>mode</a> is
"<code>navigate</code>".

<p class="note no-backref">These kind of <a for=/>responses</a> are eventually followed by a
"final" <a for=/>response</a>.
<p class=note>The following step might trigger other fetches. The assertion
annevk marked this conversation as resolved.
Show resolved Hide resolved
is to avoid potential CORS breakage.

<li><p><a>Queue a fetch task</a> to run <var>fetchParams</var>'s
<a for="fetch params">process early hints response</a>, with <a for=/>headers</a>.
annevk marked this conversation as resolved.
Show resolved Hide resolved
</ol>

<li><p><a for=iteration>Continue</a>.

<p class="note no-backref">These kind of <a for=/>responses</a> are eventually followed by
a "final" <a for=/>response</a>.
</ol>

<li><p><a for=iteration>Break</a>.
</ol>
</li>
</ul>

<p class=note>The exact layering between Fetch and HTTP still needs to be sorted through and
Expand Down