Re: [whatwg/fetch] Add support for blob range requests (PR #1520)

@annevk commented on this pull request.



> +       <var>type</var>) ยป.
+      </ol>
+
+     <li>
+      <p>Otherwise:
+
+      <ol>
+       <li><p>Set <var>response</var>'s <a for=response>range-requested flag</a>.
+
+       <li><p>Let <var>rangeHeader</var> be the result of <a for="header list">getting</a>
+       `<code>Range</code>` from <var>request</var>'s <a for=request>header list</a>.
+       <!-- Range should be added to the headers defined in HTTP extensions. See
+            https://github.com/whatwg/fetch/issues/1553 for future work -->
+
+       <li><p>Let <var>rangeValue</var> be the result of <a>parsing a single range header value</a>
+       given <var>rangeHeader</var>.

```suggestion
       given <var>rangeHeader</var> and true.
```
I think this is the only change needed here. But if you could gloss it over one last time that would be appreciated.

I suspect we'll land this somewhere next week given the Bikeshed issues.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1520#pullrequestreview-1303460935

You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1520/review/1303460935@github.com>

Received on Friday, 17 February 2023 13:49:15 UTC