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

add firstRessource timers #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add firstRessource timers #66

wants to merge 1 commit into from

Conversation

utix
Copy link
Contributor

@utix utix commented Jun 7, 2018

Add information for the first ressource (mainly the .html file):

  • dns
  • ssl
  • firstByte

Add information for the first ressource (mainly the .html file):
 * dns
 * ssl
 * firstByte
@patrickhulce
Copy link
Collaborator

Thanks for the PR @utix!

However, this repo is focused on providing speed index and perceptual speed index metrics. I'm concerned that by expanding the scope to include other types of information we lose that focus and begin to overlap (potentially inconsistently) with consuming tools. Perhaps other general performance trace processing tools might help you here instead, maybe lighthouse?

@utix
Copy link
Contributor Author

utix commented Jun 7, 2018

This metrics are key metrics to improve the speed index.
for example without ssl reuse you can loose times
When you miss you cache, the firstByte is degraded.
This metrics can quickly explain a degradation of your speedIndex

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

Successfully merging this pull request may close these issues.

2 participants