- 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