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

LH: Capture preconnect limitations #10038

Open
addyosmani opened this issue Nov 27, 2019 · 4 comments
Open

LH: Capture preconnect limitations #10038

addyosmani opened this issue Nov 27, 2019 · 4 comments

Comments

@addyosmani
Copy link
Member

tl;dr Developers should include preconnects in the order they're to be utilized

Chrome currently places a limit of 3 in-flight preconnects so developers don't need to compute how many hosts to preconnect to. This means developers connecting to a large number of hosts is WAI, but I don't believe most know the order of the preconnect origins is kinda important.

It may be worth Lighthouse include text in our docs to make it clear Chrome doesn't preconnect to all origins at the same time. So, developers should include preconnects in the order they are expected to be utilized.

Initially marking as a docs issue. We may want to factor this into the preconnect audit text if it doesn't get too wordy... :)

Confidence in this: confirmed with Tarun and Yoav
Source: https://cs.chromium.org/chromium/src/chrome/browser/predictors/preconnect_manager.h?rcl=226607d28a8085ffa3199f66954ee14c3e0736fb&l=138

Related fun tweet:
https://twitter.com/yoavweiss/status/1197551558071898113

@connorjclark
Copy link
Collaborator

https://bugs.chromium.org/p/chromium/issues/detail?id=962693

We should run a quick experiment to restrict preconnect to only origins that host render blocking content.

@paulirish
Copy link
Member

related #9894 #9903

@paulirish
Copy link
Member

and 2 year old discussion here #3106 (comment)

@connorjclark
Copy link
Collaborator

I think we still need to tweak docs? #9903 fixed this is by emitting a warning when too many preconnects happen

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

3 participants