Tencent / tquic

A high-performance, lightweight, and cross-platform QUIC library
https://tquic.net/
Other
1.02k stars 82 forks source link

Fix the problem for occasionally experiences connection loss due to Idle timeout #317

Closed XYT874264076 closed 1 month ago

XYT874264076 commented 1 month ago

When we run example demo tquic_client to send requests towards tquic_server with a single connection and a single coexistence stream, the client occasionally experiences connection loss due to Idle timeout when sending more than 10 requests. Upon investigation, we found that the Timer::Pacer in the Connection.TimeTable was not being updated in a timely manner to the Endpoint.TimeQueue. This caused the function endpoint.timeout() to ignore the Pacer timer, leading to the Idle timeout issue. This problem has now been fixed.

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.92%. Comparing base (6e35e18) to head (ecdae19).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #317 +/- ## =========================================== - Coverage 95.30% 94.92% -0.38% =========================================== Files 50 50 Lines 33598 33601 +3 =========================================== - Hits 32019 31896 -123 - Misses 1579 1705 +126 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.