- From: Anne van Kesteren <notifications@github.com>
- Date: Wed, 24 Feb 2021 00:21:11 -0800
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Wednesday, 24 February 2021 08:21:24 UTC
@annevk commented on this pull request. This looks good to me, except for the one missing parameter. I have some editorial nits I can push in a fixup commit. > + <li><p>Return a <a>network error</a>. + </ol> + + <li> + <p>If <var>actualResponse</var>'s <a for=response>status</a> is 421, + <var>request</var>'s <a for=request>body</a> is non-null, and + <var>request</var>'s <a for=request>body</a>'s <a for=body>source</a> is + null, then return a <a>network error</a>. + + <li> + <p>Set <var>bypassConnectionPool</var> to true. + + <li> + <p>Set <var>response</var> to the result of running + <a>HTTP-network-or-cache fetch</a> given <var>fetchParams</var> and + <var>bypassConnectionPool</var>. This also needs to pass isAuthenticationFetch. Not sure what value it should be. -- 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-597205830
Received on Wednesday, 24 February 2021 08:21:24 UTC