search

expressjs / response-time

Response time header for node.js
MIT License
463 stars 68 forks source link

Option to make use of Server Timing header #16

Closed ojhaujjwal closed 5 years ago

ojhaujjwal commented 5 years ago

This post gives the overview of the advantages of Server Timing and how to use it.

I think we can provide an option to use this header instead for sending response time.

dougwilson commented 5 years ago

Hi! There is certainly a lot going on on that page. It's hard to say if it's a good idea or noy, but at the very least the only way an overhaul is really going to happen is if someone makes a pull request with ssid changes, ss I don't intend to make them otherwise.

dougwilson commented 5 years ago

There is already an express middleware that does this: https://www.npmjs.com/package/server-timing

So it would seem there is no reason to duplicate that here.