From 8e202987d5c6f3e4566e5b3a63899ee7df7bcbdd Mon Sep 17 00:00:00 2001 From: Emanuele Cesena Date: Wed, 22 Jan 2025 15:57:16 -0600 Subject: [PATCH 1/2] quic: increase timeout and keep alive --- sdk/quic-definitions/src/lib.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sdk/quic-definitions/src/lib.rs b/sdk/quic-definitions/src/lib.rs index cb3385991e3242..45096c9dacd2d0 100644 --- a/sdk/quic-definitions/src/lib.rs +++ b/sdk/quic-definitions/src/lib.rs @@ -16,8 +16,8 @@ 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); +pub const QUIC_MAX_TIMEOUT: Duration = Duration::from_secs(60); +pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(45); // Based on commonly-used handshake timeouts for various TCP // applications. Different applications vary, but most seem to From 381b57fe686061e43c6b9bbfda8d58f8de43b0b9 Mon Sep 17 00:00:00 2001 From: Emanuele Cesena Date: Wed, 22 Jan 2025 16:50:50 -0600 Subject: [PATCH 2/2] quic: add comment to QUIC_MAX_TIMEOUT --- sdk/quic-definitions/src/lib.rs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sdk/quic-definitions/src/lib.rs b/sdk/quic-definitions/src/lib.rs index 45096c9dacd2d0..8e815f9457f89d 100644 --- a/sdk/quic-definitions/src/lib.rs +++ b/sdk/quic-definitions/src/lib.rs @@ -16,6 +16,10 @@ pub const QUIC_TOTAL_STAKED_CONCURRENT_STREAMS: usize = 100_000; // forwarded packets from staked nodes. pub const QUIC_MAX_STAKED_CONCURRENT_STREAMS: usize = 512; +// 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); pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(45);