Re: [whatwg/fetch] Add HTTP OWS handling to single range header parsing (PR #1564)

@annevk commented on this pull request.

Thanks, that makes sense as we don't support multiple values here. One nit and with that this is good to go.

> @@ -1292,28 +1292,45 @@ run these steps:
 
 <div algorithm>
 <p>To <dfn id=simple-range-header-value>parse a single range header value</dfn> from a
-<a>byte sequence</a> <var>value</var>, run these steps:
+<a>byte sequence</a> <var>value</var> and a boolean <var>disallowWhitespace</var>, run these steps:

Since it's a required argument, perhaps we should invert this to not have a double negative.

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

Message ID: <whatwg/fetch/pull/1564/review/1301376603@github.com>

Received on Thursday, 16 February 2023 12:53:33 UTC