Skip to content

Correct TTFB breakdown#47

Merged
nucliweb merged 1 commit into
nucliweb:mainfrom
tunetheweb:patch-1
Apr 12, 2024
Merged

Correct TTFB breakdown#47
nucliweb merged 1 commit into
nucliweb:mainfrom
tunetheweb:patch-1

Conversation

@tunetheweb

Copy link
Copy Markdown
Contributor

Makes the following changes:

  • Redirect time cannot be accurate measured so renames it to "Redirect and waiting duration".
  • "Redirect and waiting duration" should be measures from workerStart OR fetchStart to handle non-SW pages.
  • All times should be clamped to 0 after subtracting activationStart to accurately measure TTFB from activation for prerendered pages (i.e. from when user clicks rather than from when browser starts prerendering).
  • This is often a gap from connectEnd to requestStarts which is currently being attributed to SSL time. Make the following changes:
    • Change SSL duration to measure from secureConnectionStart to connectEnd to more accurately measure this.
    • Measure request from connectEnd rather than requestStart so the gap is included in request time instead.
    • This is necessary as service worker requests do not allow measurement of TCP or SSL time so they should be 0 so better to attribute to request time than SSL time.
  • Changes all "times" to "durations" to make clear they are spans of time and not actual times.

@vercel

vercel Bot commented Apr 12, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
webperf-snippets ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 12, 2024 1:21pm

@vercel

vercel Bot commented Apr 12, 2024

Copy link
Copy Markdown

@tunetheweb is attempting to deploy a commit to the Joan Len's projects Team on Vercel.

A member of the Team first needs to authorize it.

@nucliweb nucliweb left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you @tunetheweb 🙌

@nucliweb nucliweb merged commit 92e7073 into nucliweb:main Apr 12, 2024
@tunetheweb tunetheweb deleted the patch-1 branch April 12, 2024 13:37
nucliweb added a commit that referenced this pull request Feb 25, 2026
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