Skip to content

Commit

Permalink
Have HTTP-redirect fetch call main fetch with recursive=false sometimes
Browse files Browse the repository at this point in the history
Otherwise a navigation wouldn't get callbacks from processing its redirect.

Closes #1629.
  • Loading branch information
noamr committed Apr 24, 2023
1 parent 8f10983 commit ca10f49
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion fetch.bs
Original file line number Diff line number Diff line change
Expand Up @@ -5352,8 +5352,20 @@ run these steps:
<li><p>Invoke <a>set <var>request</var>'s referrer policy on redirect</a> on <var>request</var> and
<var>actualResponse</var>. [[!REFERRER]]

<li><p>Let <var>recursive</var> be true.

<li>
<p>If <var>request</var>'s <a for=request>redirect mode</a> is "<code>manual</code>", then:

<ol>
<li><p>Assert: <var>request</var>'a <a for=request>mode</a> is "<code>navigate</code>".

<li><p>Set <var>recursive</var> to false.
</ol>

<li>
<p>Return the result of running <a>main fetch</a> given <var>fetchParams</var> and true.
<p>Return the result of running <a>main fetch</a> given <var>fetchParams</var> and
<var>recursive</var>.

<p class=note>This has to invoke <a>main fetch</a> to get <var>request</var>'s
<a for=request>response tainting</a> correct.
Expand Down

0 comments on commit ca10f49

Please sign in to comment.