Re: [whatwg/fetch] Add HTTP fetch step on 421. (#1141)

@yutakahirano approved this pull request.



>  steps:
 <!-- http3Only and dedicated have been added for WebTransport -->
 
 <ol>
  <li>
-  <p>If <var>dedicated</var> is false, then:
+  <p>If <var>bypassConnectionPool</var> is false or if <var>dedicated</var> is false, then:

We don't need the second "if".

> @@ -4658,6 +4660,35 @@ steps. They return a <a for=/>response</a>.
    <var>fetchParams</var>.
   </ol>
 
+ <li>
+  <p>If the following are true
+
+  <ul class=brief>
+   <li><p><var>response</var>'s <a for=response>status</a> is 421

Please place a newline below.

Please see https://github.com/whatwg/fetch#formatting for the style.
Ditto below.

> @@ -4658,6 +4660,35 @@ steps. They return a <a for=/>response</a>.
    <var>fetchParams</var>.
   </ol>
 
+ <li>
+  <p>If the following are true

I think

```
If all of the following are true
```

is better.


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