- From: Noam Rosenthal <notifications@github.com>
- Date: Mon, 24 Apr 2023 01:07:06 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 24 April 2023 08:07:12 UTC
@noamr 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>". Mainly because that assert is potentially more confusing than helpful and I wasn't sure about it. Should we keep it or throw it? I could go either way. -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/fetch/pull/1638#discussion_r1174927297 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/fetch/pull/1638/review/1397406184@github.com>
Received on Monday, 24 April 2023 08:07:12 UTC