Skip to content

[http] discard web socket connection without handler#22183

Merged
linev merged 1 commit into
root-project:masterfrom
linev:http_regect_ws
May 13, 2026
Merged

[http] discard web socket connection without handler#22183
linev merged 1 commit into
root-project:masterfrom
linev:http_regect_ws

Conversation

@linev

@linev linev commented May 8, 2026

Copy link
Copy Markdown
Member

If no any handler was found - http request associated with specified web socket can be directly discarded.

Before connection was established - but was not handled in the ROOT.
Just was consuming resources - civetweb thread.

If no any handler was found - http request associated with specified web socket can be directly  discarded.
Connection was established - but was not handled in the ROOT.
@linev linev requested review from bellenot and dpiparo May 8, 2026 06:27
@linev linev self-assigned this May 8, 2026
@dpiparo dpiparo requested a review from jblomer May 8, 2026 06:45
@dpiparo

dpiparo commented May 8, 2026

Copy link
Copy Markdown
Member

Adding @jblomer in the reviewers pool.

@github-actions

github-actions Bot commented May 8, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 13h 33m 25s ⏱️
 3 850 tests  3 849 ✅ 0 💤 1 ❌
76 031 runs  76 030 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 01eb7df.

@linev linev merged commit 5f57840 into root-project:master May 13, 2026
31 of 33 checks passed
@linev linev deleted the http_regect_ws branch May 13, 2026 05:25
@linev

linev commented May 13, 2026

Copy link
Copy Markdown
Member Author

/backport to 6.40, 6.38, 6.36, 6.32, 6.30

@root-project-bot

Copy link
Copy Markdown

Preparing to backport PR #22183 to branches 6.40, 6.38, 6.36, 6.32 and 6.30 requested by linev

@linev

linev commented May 19, 2026

Copy link
Copy Markdown
Member Author

/backport to 6.28, 6.26

@root-project-bot

Copy link
Copy Markdown

Preparing to backport PR #22183 to branches 6.28 and 6.26 requested by linev

@root-project-bot

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants