Disable the aiohttp total timeout on the panel SSE stream (task 6c4b0fa4) #298

Merged
approver_agent merged 1 commits from task/sse-disable-total-timeout_6c4b0fa4-8b7e-4fd0-822c-f93e87d4f2a0 into dev 2026-10-06 09:12:41 -04:00
Member

Prevents the ~300s aiohttp default total timeout from cutting the healthy panel SSE stream (which, during the reconnect window, drops transitions). Lazy aiohttp import; per-request timeout=ClientTimeout(total=None, sock_connect=10).

Prevents the ~300s aiohttp default total timeout from cutting the healthy panel SSE stream (which, during the reconnect window, drops transitions). Lazy aiohttp import; per-request timeout=ClientTimeout(total=None, sock_connect=10).
dev_agent added 1 commit 2026-10-06 08:59:23 -04:00
Home Assistant's shared session defaults to ClientTimeout(total=300),
which cut a healthy event stream every five minutes; transitions fired
during the reconnect window are not replayed. The stream request now
passes ClientTimeout(total=None, sock_connect=<client timeout>), with
aiohttp imported lazily so the client stays importable without it.
Tests run the stream path against a stand-in aiohttp module fixture.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
review_agent left a comment
Member

Independent Codex gpt-5.6-terra review completed for this exact commit.

Independent Codex gpt-5.6-terra review completed for this exact commit.
approver_agent merged commit 2939bede08 into dev 2026-10-06 09:12:41 -04:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Maniacs_Mansion/NestQuest#298