Skip to content
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

Server-Timing is too limited in scope, rename to Server-Metrics #77

Open
plutora-steve-g opened this issue Aug 26, 2020 · 2 comments
Open

Comments

@plutora-steve-g
Copy link

plutora-steve-g commented Aug 26, 2020

Server-Timing is a one-trick pony. It would be nice if was a header that could express any server metric rather than just times, which are interesting, but only part of the story if you're trying to analyse what's going on in server-land. To that end I propose renaming SERVER-TIMING to SERVER-METRICS so it's future proof.

As an example, I've been trying to use Server-Timing to express other internal server metrics apart from just times:

db;dur=100;avg=10;stddev=5.3;count=10;sp=3;ef=7;deadlockRetries=1
cache;hit=10;miss=2;duration=100;avg=10;stddev=4.5

To me Server-Timing is client-focused, not server-focused. It's been driven from the client POV not the server POV -- duration is just one of many metrics that are useful in troubleshooting and development of a server. Exposing arbitrary metrics makes it possible to answer the question, "Why Is It Slow", which is far more useful than the statement, "It Is Slow".

@noamr
Copy link
Contributor

noamr commented Aug 15, 2022

I'm not sure there's enough steam behind renaming server timing at this point. Feel free to gather more support.

@josephwynn-sc
Copy link

I'm not necessarily voting for a rename, but just wanted to raise my hand and say that at SpeedCurve we're supporting customers sending all sorts of data through the server-timing header. It's misuse/abuse of the spec, but essentially we're allowing arbitrary numeric or string values in the desc field. It would be nice if the spec and API had first-class support for data types other than timing durations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants