Skip to content

quic: increase timeout and keep alive #4585

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

Merged
lijunwangs merged 4 commits into from
Jan 24, 2025
Merged

quic: increase timeout and keep alive #4585

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8e20298
quic: increase timeout and keep alive
0x0ece Jan 22, 2025
381b57f
quic: add comment to QUIC_MAX_TIMEOUT
0x0ece Jan 22, 2025
88aa4d8
Merge branch 'master' into quic-timeout
lijunwangs Jan 23, 2025
dfd19d8
Merge branch 'master' into quic-timeout
lijunwangs Jan 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions sdk/quic-definitions/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,12 @@ pub const QUIC_TOTAL_STAKED_CONCURRENT_STREAMS: usize = 100_000;
// forwarded packets from staked nodes.
pub const QUIC_MAX_STAKED_CONCURRENT_STREAMS: usize = 512;

pub const QUIC_MAX_TIMEOUT: Duration = Duration::from_secs(2);
pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(1);
// Connection idle timeout, and keep alive.
// Quic will close the connection after QUIC_MAX_TIMEOUT,
// and send a ping every QUIC_KEEP_ALIVE.
// These shouldn't be too low to avoid unnecessary ping traffic.
pub const QUIC_MAX_TIMEOUT: Duration = Duration::from_secs(60);
Comment thread
0x0ece marked this conversation as resolved.
pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(45);

// Disable Quic send fairness.
// When set to false, streams are still scheduled based on priority,
Expand Down