Increase QUIC idle timeout to 30s and keep-alive to 5s (#1221)

Co-authored-by: Claude <noreply@anthropic.com>

Author: kixelated

rs/moq-native/src/noq.rs

@@ -23,8 +23,8 @@ impl NoqClient {
 		let socket = std::net::UdpSocket::bind(config.bind).context("failed to bind UDP socket")?;
 
 		let mut transport = noq::TransportConfig::default();
-		transport.max_idle_timeout(Some(time::Duration::from_secs(10).try_into().unwrap()));
-		transport.keep_alive_interval(Some(time::Duration::from_secs(4)));
+		transport.max_idle_timeout(Some(time::Duration::from_secs(30).try_into().unwrap()));
+		transport.keep_alive_interval(Some(time::Duration::from_secs(5)));
 		transport.mtu_discovery_config(None); // Disable MTU discovery
 
 		let max_streams = config.max_streams.unwrap_or(crate::DEFAULT_MAX_STREAMS);
@@ -148,8 +148,8 @@ pub(crate) struct NoqServer {
 impl NoqServer {
 	pub fn new(config: ServerConfig) -> anyhow::Result<Self> {
 		let mut transport = noq::TransportConfig::default();
-		transport.max_idle_timeout(Some(Duration::from_secs(10).try_into().unwrap()));
-		transport.keep_alive_interval(Some(Duration::from_secs(4)));
+		transport.max_idle_timeout(Some(Duration::from_secs(30).try_into().unwrap()));
+		transport.keep_alive_interval(Some(Duration::from_secs(5)));
 		transport.mtu_discovery_config(None); // Disable MTU discovery
 
 		let max_streams = config.max_streams.unwrap_or(crate::DEFAULT_MAX_STREAMS);

rs/moq-native/src/quinn.rs

@@ -22,8 +22,8 @@ impl QuinnClient {
 
 		// TODO Validate the BBR implementation before enabling it
 		let mut transport = quinn::TransportConfig::default();
-		transport.max_idle_timeout(Some(time::Duration::from_secs(10).try_into().unwrap()));
-		transport.keep_alive_interval(Some(time::Duration::from_secs(4)));
+		transport.max_idle_timeout(Some(time::Duration::from_secs(30).try_into().unwrap()));
+		transport.keep_alive_interval(Some(time::Duration::from_secs(5)));
 		transport.mtu_discovery_config(None); // Disable MTU discovery
 
 		let max_streams = config.max_streams.unwrap_or(crate::DEFAULT_MAX_STREAMS);
@@ -149,8 +149,8 @@ impl QuinnServer {
 		// Enable BBR congestion control
 		// TODO Validate the BBR implementation before enabling it
 		let mut transport = quinn::TransportConfig::default();
-		transport.max_idle_timeout(Some(Duration::from_secs(10).try_into().unwrap()));
-		transport.keep_alive_interval(Some(Duration::from_secs(4)));
+		transport.max_idle_timeout(Some(Duration::from_secs(30).try_into().unwrap()));
+		transport.keep_alive_interval(Some(Duration::from_secs(5)));
 		transport.mtu_discovery_config(None); // Disable MTU discovery
 
 		let max_streams = config.max_streams.unwrap_or(crate::DEFAULT_MAX_STREAMS);