aboutsummaryrefslogtreecommitdiff
path: root/.github/pull_request_template.md
diff options
context:
space:
mode:
authoramankumarsingh77 <amankumarsingh7702@gmal.com>2026-04-30 13:09:27 +0530
committeramankumarsingh77 <amankumarsingh7702@gmal.com>2026-04-30 13:09:27 +0530
commitb6d752222947229112132ac0fbd87f355fc78974 (patch)
treec3c9424f25b5b87945079dc653ec72b0b510022d /.github/pull_request_template.md
parent63b16957587fe20566129ffc5191f66f24a70872 (diff)
fix: stop Chrome remote-debugging consent popup storm
Previously a single consent-pending socket could spawn up to 5 stacked 'Allow remote debugging?' dialogs. Three causes: - onopen reset _reconnectAttempts, but Chrome accepts the WS upgrade before the user clicks Allow, so a held socket looked healthy. - ensureAlive's health probe used the full 15s CDP timeout; while the user was reading the dialog the probe would time out, trigger a stop()+start(), and a fresh popup would stack on top. - onclose auto-reconnected unconditionally even when consent had never been granted on that connection. Now a connection is only considered healthy once the first CDP roundtrip completes (attachFirstPage). ensureAlive does a 2s short probe before tearing down the socket. onclose bails out of auto- reconnect when consent was never observed, so a denied dialog surfaces a clean error instead of looping.
Diffstat (limited to '.github/pull_request_template.md')
0 files changed, 0 insertions, 0 deletions