Add connection timeout to DownloadManager session.get()

Richard Taylor · claude · Richard Taylor · commit 6ca87834ea42 · 2026-03-29T09:07:07.000+01: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: 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
@@ -115,7 +115,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) as response:
+            async with session.get(url, headers=headers, 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}")
