Skip to content

Task 5 homework#3

Open
ArtsiomMusin wants to merge 3 commits intospajic:masterfrom
ArtsiomMusin:task_5_homework
Open

Task 5 homework#3
ArtsiomMusin wants to merge 3 commits intospajic:masterfrom
ArtsiomMusin:task_5_homework

Conversation

@ArtsiomMusin
Copy link

  1. Скриншот с работающим server-push
    image
node_modules/.bin/http2-push-detect https://localhost
Receiving pushed resource: /assets/bell.svg
Receiving pushed resource: /assets/menu.svg
Receiving pushed resource: /assets/connect.svg
Receiving pushed resource: /assets/stack.svg
Receiving pushed resource: /assets/lightning.svg

  1. Наблюдения о работе HTTP/2.

Как показалось HTTP/1.1 работает медленнее чем HTTP/2. Часто наблюдались блокировки запросов на HTTP/1.1 при загрузке картинок, но после включения HTTPS/2 загрузка картинок ни разу не блокировалась другими запросами.
Что в свою очередь сократило загрузки некоторых запросов с 300ms до 150-200ms.

При включенном server-push, загрузка картинок пропала из вкладки Networks вообще для тех svg файлов,что были добавлены в push_headers.
Однако, возможно server-push имеет смысл для часто повторяемых или для большего количества картинок, так как прогоны ab до и после не показали ускорения общего времени загрузки сайта, а даже немного замедлили на 100-300ms.

  1. Анализ https://dev.to на webpagetest.

Результаты тестирования - http://webpagetest.org/result/190427_PX_add779b38e73f1adb5a2a072473e6cbd/

Проверил то, что dev.to уже использует HTTP/2. Но нет ничего для server-push:
node_modules/.bin/http2-push-detect https://dev.to - ничего не показало.

Имеет смысл на проде dev.to использовать server-push хотя бы для тех же svg файлов. К примеру, lightning.svg требует около 600ms.

Еще одна интересное место - загрузка profile images для пользователей. Webpagetest показал, что около 2 секунд занимает загрузка только эти картинок. Возможно здесь server-push поможет ускорить загрузку всего сайта, так как одна картинка профайла загружается около 300ms, то можно попробовать выйграть около 1.5 секунды.

Copy link
Owner

@spajic spajic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool

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