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.

Sign up for GitHub

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

Jump to bottom

Display "Total Connection Time" in Request Details Modal #1755

Closed
screenspan opened this issue Feb 16, 2022 · 3 comments · Fixed by #1765
Closed

Display "Total Connection Time" in Request Details Modal #1755

screenspan opened this issue Feb 16, 2022 · 3 comments · Fixed by #1765
Assignees
@tkadlec
Labels
Type: Enhancement An improvement to WebPageTest

Comments

@screenspan
Copy link

Feature Request Summary

I'd like to see the "Total Connection Time" (DNS Lookup + Initial Connection + SSL Negotiation) in the Request Details Modal to more quickly figure out what might be saved by moving a resource to an open connection.

Screen Shot 2022-02-16 at 20 19 15

Screen Shot 2022-02-16 at 20 21 58

What's the motivation or use case for the change/feature?

I often come across (very often 3P) requests that incur extra connection costs. It would be helpful to quickly see just how long the connection took to establish.
@screenspan screenspan added the Type: Enhancement An improvement to WebPageTest label Feb 16, 2022
@tkadlec
Copy link
Contributor

tkadlec commented Feb 16, 2022

File this one under "things that have annoyed Tim and yet it never crossed his mind to just fix it".

Good idea. :) We can do this.

@tkadlec tkadlec self-assigned this Feb 16, 2022
tkadlec added a commit that referenced this issue Feb 23, 2022
@tkadlec
This adds a Total Connection Time to the request details dialog to ma…
a24cdd4 
…ke it easier to see the combined impact of DNS+TCP+SSL.

While we're in here, I also broke up the long list of request details to give it some categorization and, hopefully, make it easier to scan.

Closes #1755.
@tkadlec tkadlec mentioned this issue Feb 23, 2022
Merged
@screenspan
Copy link
Author

Awesome! Thanks a lot!

@tkadlec
Copy link
Contributor

tkadlec commented Mar 1, 2022

Thanks for filling! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tkadlec tkadlec

Labels
Type: Enhancement An improvement to WebPageTest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Total Connection Time to the Request Details Dialog catchpoint/WebPageTest
2 participants
@tkadlec @screenspan