Skip to content

[release/9.0-staging] Deny unmasked frame receive for WebSocket Server#123661

Merged
wfurt merged 3 commits intorelease/9.0-stagingfrom
backport/pr-123485-to-release/9.0-staging
Mar 13, 2026
Merged

[release/9.0-staging] Deny unmasked frame receive for WebSocket Server#123661
wfurt merged 3 commits intorelease/9.0-stagingfrom
backport/pr-123485-to-release/9.0-staging

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 27, 2026

Backport of #123485 to release/9.0-staging

/cc @liveans

Increasing RFC compliance for WebSocket

Customer Impact

RFC compliance

Regression

No

Testing

Manual verification + automated tests

Risk

Low, the change only affects non‑compliant WebSocket clients sending unmasked frames, which is explicitly disallowed by RFC 6455. No behavior change is expected for compliant clients.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@liveans
Copy link
Member

liveans commented Mar 12, 2026

Approved by Tactics on 3/11 over email.

@liveans liveans added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Mar 12, 2026
@wfurt
Copy link
Member

wfurt commented Mar 13, 2026

/ba-g "known errors"

@wfurt wfurt merged commit 309aa7f into release/9.0-staging Mar 13, 2026
77 of 82 checks passed
@wfurt wfurt deleted the backport/pr-123485-to-release/9.0-staging branch March 13, 2026 00:30
wfurt pushed a commit to jozkee/runtime that referenced this pull request Mar 13, 2026
dotnet#123661)

Backport of dotnet#123485 to release/9.0-staging

/cc @liveans

Increasing RFC compliance for WebSocket

## Customer Impact

RFC compliance

## Regression

No

## Testing

Manual verification + automated tests

## Risk

Low, the change only affects non‑compliant WebSocket clients sending
unmasked frames, which is explicitly disallowed by RFC 6455. No behavior
change is expected for compliant clients.

---------

Co-authored-by: Ahmet İbrahim Aksoy <[email protected]>
Co-authored-by: Copilot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Net Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants