Use milliseconds in header

tuupola / server-timing-middleware

PSR-7 & PSR-15 middleware to add the Server-Timing header

MIT License

198 stars 9 forks source link

Use milliseconds in header #2

Closed jyggen closed 7 years ago

jyggen commented 7 years ago

The current code, which uses seconds in the header, results in some strange timings in (at least) Chrome's dev tools. It seems like milliseconds is what's expected.

Before (using $seconds): screen shot 2017-05-11 at 16 34 37

After (using $timing): screen shot 2017-05-11 at 16 33 47

tuupola commented 7 years ago

I do not see this behaviour with current code. Which Chrome version you are testing with? PHP version and server OS might be useful too.

tuupola commented 7 years ago

Updated my Chrome. Seems behaviour has changed between 57 and 58. Thanks for the heads up!

tuupola commented 7 years ago

Related Chromium issue: https://codereview.chromium.org/2689833002