- From: Anne van Kesteren <notifications@github.com>
- Date: Mon, 04 Jun 2018 05:22:39 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 4 June 2018 12:23:02 UTC
annevk commented on this pull request.
> @@ -3652,8 +3652,20 @@ Range Requests</cite>. [[HTTP-RANGE]] However, this is not widely supported by b
</ol>
<li>
- <p>Modify <var>httpRequest</var>'s
- <a for=request>header list</a> per HTTP.
+ <p>If <var>httpRequest</var>'s <a for=request>header list</a> <a for="header list">contains</a>
+ `<code>Range</code>`, then <a for="header list">append</a>
+ `<code>Accept-Encoding</code>`/`<code>identity</code>` to <var>httpRequest</var>'s
+ <a for=request>header list</a>.
+
+ <p class="note no-backref"><a href="https://jakearchibald.github.io/accept-encoding-range-test/">Many
+ servers</a> mistakenly ignore `<code>Range</code>` headers if a non-identity encoding is
No newline here. Only HTML allows newlines in inlines. I can fix this up for you before landing though.
--
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/751#pullrequestreview-125548728
Received on Monday, 4 June 2018 12:23:02 UTC