Add connection timeout to DownloadManager session.get() (#98)

bitcoin3us · Richard Taylor · claude · web-flow · commit 787cec873dfd · 2026-03-30T13:05:11.000+02:00
The HTTP connection (session.get) had no timeout, only individual chunk
reads did. If the TCP connection or TLS handshake stalled, the entire
async balance fetch loop would hang forever. This was observed on ESP32
devices where intermittent WiFi issues caused session.get() to never
return, freezing the wallet UI on "Connecting to backend".

Uses the existing _CHUNK_TIMEOUT_SECONDS (10s) for the connection timeout.

Co-authored-by: Richard Taylor &lt;RT@MacBookPro.lan&gt;
Co-authored-by: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/internal_filesystem/lib/mpos/net/download_manager.py b/internal_filesystem/lib/mpos/net/download_manager.py
@@ -114,7 +114,7 @@ async def _download_url_async(cls, url, outfile=None, total_size=None,
             if headers is None:
                 headers = {}
             
-            async with session.get(url, headers=headers, ssl=sslctx) as response:
+            async with session.get(url, headers=headers, ssl=sslctx, timeout=_CHUNK_TIMEOUT_SECONDS) as response:
                 if response.status < 200 or response.status >= 400:
                     print(f"DownloadManager: HTTP error {response.status}")
                     raise RuntimeError(f"HTTP {response.status}")
