feat: improve task cancellation and error logging in DomainFronter

main.py

@@ -310,6 +310,12 @@ async def _run(config):
         await server.start()
     finally:
         await server.stop()
+        # Cancel any tasks that leaked through (e.g. fire-and-forget pool tasks).
+        stray = [t for t in asyncio.all_tasks() if t is not asyncio.current_task()]
+        for t in stray:
+            t.cancel()
+        if stray:
+            await asyncio.gather(*stray, return_exceptions=True)
 
 
 if __name__ == "__main__":

src/relay/domain_fronter.py

@@ -511,7 +511,7 @@ class DomainFronter:
                 reader, writer, created = self._pool.pop()
                 if (now - created) < self._conn_ttl and not reader.at_eof():
                     # Eagerly replace the connection we just took
-                    asyncio.create_task(self._add_conn_to_pool())
+                    self._spawn(self._add_conn_to_pool())
                     return reader, writer, created
                 try:
                     writer.close()
@@ -954,13 +954,19 @@ class DomainFronter:
             log.info("H2 multiplexing active — one conn handles all requests")
         except Exception as e:
             self._record_h2_failure(e)
-            log.warning("H2 connect failed (%s), using H1 pool fallback", e)
+            log.warning(
+                "H2 connect failed (%s: %s), using H1 pool fallback",
+                type(e).__name__,
+                e or "(no details)",
+            )
 
     async def _h2_connect_and_warm(self):
         """Connect H2, pre-warm the Apps Script container, start keepalive."""
         await self._h2_connect()
         if self._h2_available():
             self._spawn(self._prewarm_script())
+        # Always start keepalive — even on startup failure it will retry H2
+        # once the cooldown expires instead of leaving H1-only permanently.
         if self._keepalive_task is None or self._keepalive_task.done():
             self._keepalive_task = self._spawn(self._keepalive_loop())
 
@@ -1015,12 +1021,24 @@ class DomainFronter:
         """Send periodic pings to keep Apps Script warm + H2 connection alive."""
         while True:
             try:
-                await asyncio.sleep(240)  # 4 minutes — saves ~90 quota hits/day vs 180s
-                                          # Google's container timeout is ~5 min idle
-                if not self._h2_available():
+                # Keep a conservative cadence to avoid any chance of this loop
+                # contending with foreground relay work.
+                await asyncio.sleep(240)
+
+                # If H2 is absent or still in cooldown, skip this tick.
+                if self._h2 is None or time.time() < self._h2_disabled_until:
+                    continue
+
+                # Reconnect in background when needed, but bound it with a
+                # timeout so recovery attempts can never stall the loop.
+                if not self._h2.is_connected:
                     try:
-                        await self._h2.reconnect()
+                        await asyncio.wait_for(
+                            self._h2.reconnect(),
+                            timeout=max(self._tls_connect_timeout, 8.0),
+                        )
                         self._record_h2_success()
+                        log.info("H2 re-established after failure")
                     except Exception as exc:
                         self._record_h2_failure(exc)
                         continue
@@ -1813,7 +1831,8 @@ class DomainFronter:
     @classmethod
     def _is_static_asset_url(cls, url: str) -> bool:
         path = urlparse(url).path.lower()
-        return any(path.endswith(ext) for ext in cls._STATIC_EXTS)
+        # Also match versioned paths like /script.js/v3a4b… or /font.woff2/hash
+        return any(path.endswith(ext) or f"{ext}/" in path for ext in cls._STATIC_EXTS)
 
     @staticmethod
     def _header_value(headers: dict | None, name: str) -> str:
@@ -1947,8 +1966,8 @@ class DomainFronter:
             except Exception as e:
                 log.warning(
                     "Batch relay failed, disabling batch mode for %ds cooldown. "
-                    "Error: %s",
-                    self._batch_cooldown, e,
+                    "Error: %s: %s",
+                    self._batch_cooldown, type(e).__name__, e or "(no details)",
                 )
                 self._batch_enabled = False
                 self._batch_disabled_at = time.time()