Skip to content

Bump WiFi client connect timeout from 30s to 60s - #71

Open
eliasbakken wants to merge 1 commit into
mainfrom
enhancement/bump-wifi-client-timeout
Open

Bump WiFi client connect timeout from 30s to 60s#71
eliasbakken wants to merge 1 commit into
mainfrom
enhancement/bump-wifi-client-timeout

Conversation

@eliasbakken

Copy link
Copy Markdown
Contributor

Problem

Observed a real timeout on a 5GHz network in practice: the boot-time connection attempt hit the full 30s window and fell back to hotspot, even though the network was reachable (reconnecting manually right after succeeded within a few seconds).

5GHz scanning - particularly across DFS channels, which require slower passive scanning - can plausibly eat into a cold boot-time connect more than 30s comfortably allows.

Why 60s

The cost is asymmetric: a longer timeout just means a slightly longer boot delay in the rare case the SSID truly isn't reachable at all, but a too-short timeout means falling back to hotspot even when the network would have connected given a few more seconds - which then needs manual intervention to fix. Per #65, the hotspot is meant to be a last-resort backup, not something to hit routinely.

🤖 Generated with Claude Code

Observed a real timeout on a 5GHz network in practice: the boot-time
connection attempt hit the full 30s window and fell back to hotspot,
even though the network was reachable and reconnecting manually
succeeded moments later. 5GHz scanning (particularly across DFS
channels, which require slower passive scanning) can plausibly eat
into a cold boot-time connect more than 30s comfortably allows.

The cost is asymmetric: a longer timeout just means a slightly longer
boot delay in the rare case the SSID truly isn't reachable, but a too
-short timeout means falling back to hotspot even when the network
would have connected given a few more seconds - which then needs
manual intervention to fix, defeating the point of #65's design
(hotspot is meant to be a last-resort backup, not something to hit
routinely).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant