Re: [whatwg/fetch] Editorial: Pass recursive=false to manual navigation redirect (PR #1638)

@annevk commented on this pull request.



> @@ -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>".

This is incorrect. You can always set it to "manual" and get an opaque response at the moment. Mainly useful in service workers with navigations though.

I suggest we do something like

> Let recursive be true if ....; otherwise false.

instead.

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

Message ID: <whatwg/fetch/pull/1638/review/1397363296@github.com>