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

improve timeout measurement in the timeout test #3042

Merged
merged 3 commits into from
Mar 2, 2021

Conversation

marten-seemann
Copy link
Member

When not sending any packets, the idle timeout starts when receiving the HANDSHAKE_DONE frame (on the client side), not when the handshake completes.

@codecov
Copy link

codecov bot commented Feb 15, 2021

Codecov Report

Merging #3042 (11c5c54) into master (c72f05a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3042   +/-   ##
=======================================
  Coverage   86.20%   86.20%           
=======================================
  Files         131      131           
  Lines        9356     9356           
=======================================
  Hits         8065     8065           
  Misses        935      935           
  Partials      356      356           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c72f05a...11c5c54. Read the comment docs.

@marten-seemann marten-seemann changed the title improve timeout measumrent in the timeout test improve timeout measurement in the timeout test Feb 17, 2021
When not sending any packets, the idle timeout starts when receiving the
HANDSHAKE_DONE frame (on the client side), not when the handshake completes.
@marten-seemann marten-seemann merged commit 40bc736 into master Mar 2, 2021
@marten-seemann marten-seemann deleted the improve-timeout-test branch March 3, 2021 03:27
@aschmahmann aschmahmann mentioned this pull request May 14, 2021
71 tasks
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