diff options
| author | amankumarsingh77 <amankumarsingh7702@gmal.com> | 2026-04-30 13:09:27 +0530 |
|---|---|---|
| committer | amankumarsingh77 <amankumarsingh7702@gmal.com> | 2026-04-30 13:09:27 +0530 |
| commit | b6d752222947229112132ac0fbd87f355fc78974 (patch) | |
| tree | c3c9424f25b5b87945079dc653ec72b0b510022d /.github/pull_request_template.md | |
| parent | 63b16957587fe20566129ffc5191f66f24a70872 (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
