Re: [whatwg/fetch] Process 103 Early Hints response (PR #1404)

@annevk commented on this pull request.



> +       <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>Processing of early hints is only vetted for navigations.

This is the most significant change I made. I made it because the earlier assertion focused on "cors" while "same-origin" also exists and wasn't addressed. And it's also not clear to me that "cors" couldn't be made to work if we wanted it to work.

If this and my other changes look okay (please an explicit comment if you had a look) I think this can land.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1404#pullrequestreview-905734660
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1404/review/905734660@github.com>