Skip to content

Commit

Permalink
Make CORS-preflight fetches set the CORS flag
Browse files Browse the repository at this point in the history
Otherwise a 401 response would not necessarily result in rejection.

Fixes #741.
  • Loading branch information
annevk committed May 29, 2018
1 parent fa6f2b0 commit 9334fcb
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions fetch.bs
Original file line number Diff line number Diff line change
Expand Up @@ -4144,8 +4144,8 @@ the <a>CORS protocol</a> is understood. The so-called <a>CORS-preflight request<
successful it populates the <a>CORS-preflight cache</a> to minimize the
number of these <a lt="CORS-preflight fetch">fetches</a>.

<p>To perform a <dfn id=cors-preflight-fetch-0>CORS-preflight fetch</dfn> using <var>request</var>, run these
steps:
<p>To perform a <dfn id=cors-preflight-fetch-0>CORS-preflight fetch</dfn> using <var>request</var>,
run these steps:

<ol>
<li>
Expand Down Expand Up @@ -4190,8 +4190,7 @@ steps:
0x2C is not the way this was implemented, for better or worse.

<li><p>Let <var>response</var> be the result of performing an
<a>HTTP-network-or-cache fetch</a> using
<var>preflight</var>.
<a>HTTP-network-or-cache fetch</a> using <var>preflight</var> with the <i>CORS flag</i> set.

<li>
<p>If a <a for=cors>CORS check</a> for <var>request</var> and <var>response</var> returns success
Expand Down

0 comments on commit 9334fcb

Please sign in to comment.