-
Notifications
You must be signed in to change notification settings - Fork 342
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Private (conceptual) trailer support #980
Comments
As noted in #772 (comment) the current way it is exposed subsets what HTTP supports and therefore does not feel like a good starting point. Both for the internal and public API. Additionally, the public-facing API has not seen interest from implementers, at least for the past year and a half. Tests: web-platform-tests/wpt#20720. Closes #772. Follow-ups: #980 and #981.
What's the current implementor interest around supporting trailers, and in particular for |
I do think it would be good to revisit support for HTTP trailers in responses and requests in general, particularly in light of things like https://aws.amazon.com/blogs/aws/new-additional-checksum-algorithms-for-amazon-s3/ which make direct use of trailers in both directions. |
/cc @ddragana @mcmanus - for their opinions from the Mozilla side as an implementer as well as from Fastly as a user. From Chromium, I haven't heard of anyone interested in implementing trailers, even if just for Server-Timing. (but I'll ask around, as it's been a while) @jasnell - does your use case require Server-Timing trailer support, or trailers in general? |
I dug up #34 which contains a bunch of arguments against trailer support. I'm not sure if those are still current, as it's been a while. |
I read the thread. The arguments against mostly don't apply to However timing info was presented there as an actual valid specific use case. |
Caches are allowed to drop trailers. |
Trailers in general... but don't take it as a We Must Have It type of thing. We are looking at whether or not we could ever support that new AWS spec without resorting to non-standard extensions and it's looking most likely that we won't. |
@mnot - while true, if we were to expose Server-Timing in trailers, whether caches drop them or not would be extremely web visible.. |
I wonder what Firefox does. @ddragana? I wouldn't mind specifying that they are always dropped, but I haven't thought through the implications. |
To support something like https://w3c.github.io/server-timing/#server-timing-header-parsing-algorithm the response concept needs trailer support. I'm not sure if we need to add conceptual request trailer support as part of solving this issue as I'm not sure there are any dependencies. But any design should work for them as well, I suppose.
Per HTTP semantics trailers (a header list) can arrive "concurrently" with a message body and they can arrive multiple times (i.e., an asynchronous sequence of header lists). Server Timing does not appear to account for that.
(See #772 and #979 for prior takes on response trailer support.)
The text was updated successfully, but these errors were encountered: