Thanks to visit codestin.com
Credit goes to github.com

Skip to content

browser: prevent websocket abnormal-close deadlock#5697

Draft
inancgumus wants to merge 32 commits intomasterfrom
codex/fix/browser-websocket-abnormal-closure
Draft

browser: prevent websocket abnormal-close deadlock#5697
inancgumus wants to merge 32 commits intomasterfrom
codex/fix/browser-websocket-abnormal-closure

Conversation

@inancgumus
Copy link
Contributor

@inancgumus inancgumus commented Feb 24, 2026 •
edited
Loading

  • prevent unexpected websocket-close paths from blocking on errorCh
  • keep close signaling progressing so connection teardown can complete
  • add focused abnormal-closure tests for idle and pending-request cases

@inancgumus inancgumus requested a review from a team as a code owner February 24, 2026 20:03
@inancgumus inancgumus requested review from ankur22 and oleiade and removed request for a team February 24, 2026 20:03
@inancgumus inancgumus marked this pull request as draft February 24, 2026 20:03
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 20:09 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 20:11 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 21:28 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 21:30 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 22:18 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 24, 2026 22:20 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 00:23 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 00:26 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 00:57 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 01:02 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 01:27 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 01:29 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 04:55 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 04:57 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:08 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:10 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:25 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:26 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:39 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:40 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:53 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 05:55 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:07 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:08 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:20 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:23 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:34 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:37 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:50 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 06:51 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 07:04 — with GitHub Actions Inactive
@inancgumus inancgumus temporarily deployed to azure-trusted-signing February 25, 2026 07:06 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@inancgumus