- From: Guohui Deng <notifications@github.com>
- Date: Mon, 12 May 2025 11:21:44 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 12 May 2025 18:21:48 UTC
@guohuideng2024 commented on this pull request.
> @@ -6319,6 +6321,23 @@ optional boolean <var>forceNewConnection</var> (default false), run these steps:
<li><p>Let <var>codings</var> be the result of <a>extracting header list values</a> given
`<code>Content-Encoding</code>` and <var>response</var>'s <a for=response>header list</a>.
+ <li><p>Let <var>filteredCoding</var> be "<code>_</code>".
Yes. The CL is being reviewed right now:
https://chromium-review.googlesource.com/c/chromium/src/+/6506187
The following doc should be the one specifying the syntax restrictions on the values: https://www.rfc-editor.org/rfc/rfc8941.html and there is no restriction saying the value cannot start with "_". So, `_unkown` is better than `_` but I think it still requires reserving it with ietf. Let me email Roy to get some clarification on this.
On the other hand I think "the empty string" is more conventional than `_missing` so I fee strongly for the `the empty string`.
--
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1796#discussion_r2085211783
You are receiving this because you are subscribed to this thread.
Message ID: <whatwg/fetch/pull/1796/review/2834103374@github.com>
Received on Monday, 12 May 2025 18:21:48 UTC