Re: [whatwg/fetch] Add server-timing to timing info struct (PR #1406) from Anne van Kesteren on 2022-03-10 (public-webapps-github@w3.org from March 2022)

From: Anne van Kesteren <notifications@github.com>
Date: Thu, 10 Mar 2022 04:49:42 -0800
To: whatwg/fetch <fetch@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <whatwg/fetch/pull/1406/review/905783673@github.com>

@annevk approved this pull request.

I'd like @ddragana or @valenting to comment as well given that Firefox does support trailers here if I'm not mistaken, but this looks okay to me. I pushed some minor commits that I hope are acceptable.

I'm not entirely sure about the approach to processing this header being shared across two specifications, but I also cannot really think of a better way of doing it.



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

Message ID: <whatwg/fetch/pull/1406/review/905783673@github.com>

Received on Thursday, 10 March 2022 12:49:54 UTC