Increase idle connection timeout 10sec -> 1h (#27)

Increase idle connection timeout from default 10 seconds to 1 hour to give IPFS clients more time to query data over Bitswap. This is needed when manually adding our node to a swarm of an IPFS node, because the IPFS node doesn't keep any active substreams with us and our node closes a connection after idle_connection_timeout.

Author: dmitry-markin

Cargo.lock

@@ -16,6 +16,7 @@ serde = { version = "1.0.126" }
 serde_json = { version = "1.0.132", default-features = false }
 static_assertions = { version = "1.1" }
 try-runtime-cli = { version = "0.42" }
+tracing = { version = "0.1.41", default-features = false }
 
 [workspace]
 resolver = "2"

Cargo.toml

@@ -18,6 +18,7 @@ name = "polkadot-bulletin-chain"
 [dependencies]
 clap = { features = ["derive"], workspace = true }
 futures = { features = ["thread-pool"], workspace = true }
+tracing = { workspace = true }
 
 frame-system = { git = "https://github.com/paritytech/polkadot-sdk.git", rev = "db5e645422ccf952018a3c466a33fef477858602" }
 sc-cli = { git = "https://github.com/paritytech/polkadot-sdk.git", rev = "db5e645422ccf952018a3c466a33fef477858602" }

node/Cargo.toml

@@ -8,14 +8,17 @@ use frame_benchmarking_cli::{BenchmarkCmd, ExtrinsicFactory, SUBSTRATE_REFERENCE
 use polkadot_bulletin_chain_runtime::Block;
 use sc_cli::SubstrateCli;
 use sc_service::PartialComponents;
-use std::sync::Arc;
+use std::{sync::Arc, time::Duration};
 
 #[cfg(feature = "try-runtime")]
 use {
 	polkadot_bulletin_chain_runtime::SLOT_DURATION,
 	try_runtime_cli::block_building_info::timestamp_with_babe_info,
 };
 
+/// Log target for this file.
+const LOG_TARGET: &str = "command";
+
 impl SubstrateCli for Cli {
 	fn impl_name() -> String {
 		"Polkadot Bulletin Chain Node".into()
@@ -206,7 +209,25 @@ pub fn run() -> sc_cli::Result<()> {
 		},
 		None => {
 			let runner = cli.create_runner(&cli.run)?;
-			runner.run_node_until_exit(|config| async move {
+			runner.run_node_until_exit(|mut config| async move {
+				// Override default idle connection timeout of 10 seconds to give IPFS clients more
+				// time to query data over Bitswap. This is needed when manually adding our node
+				// to a swarm of an IPFS node, because the IPFS node doesn't keep any active
+				// substreams with us and our node closes a connection after
+				// `idle_connection_timeout`.
+				const IPFS_WORKAROUND_TIMEOUT: Duration = Duration::from_secs(3600);
+
+				if config.network.idle_connection_timeout < IPFS_WORKAROUND_TIMEOUT {
+					tracing::info!(
+						target: LOG_TARGET,
+						old = ?config.network.idle_connection_timeout,
+						overriden_with = ?IPFS_WORKAROUND_TIMEOUT,
+						"Overriding `config.network.idle_connection_timeout` to allow long-lived connections with IPFS nodes",
+
+					);
+					config.network.idle_connection_timeout = IPFS_WORKAROUND_TIMEOUT;
+				}
+
 				service::new_full::<sc_network::Litep2pNetworkBackend>(config)
 					.map_err(sc_cli::Error::Service)
 			})