Skip to content

add socket attribute to IncomingMessage#4721

Merged
anonrig merged 5 commits intomainfrom
yagiz/add-http-socket
Aug 8, 2025
Merged

add socket attribute to IncomingMessage#4721
anonrig merged 5 commits intomainfrom
yagiz/add-http-socket

Conversation

@anonrig
Copy link
Copy Markdown
Member

@anonrig anonrig commented Aug 7, 2025

Adds the missing socket attribute to IncomingMessage. After landing this, we can remove the experimental attribution from the compat flag.

@anonrig anonrig requested review from a team as code owners August 7, 2025 20:59
@anonrig anonrig requested a review from a team August 7, 2025 20:59
@anonrig anonrig requested a review from a team as a code owner August 7, 2025 20:59
@anonrig anonrig force-pushed the yagiz/add-http-socket branch 2 times, most recently from bed1bf3 to bb4362e Compare August 7, 2025 21:18
@anonrig anonrig requested review from danlapid and npaun August 7, 2025 21:20
@anonrig anonrig requested a review from jasnell August 7, 2025 21:22
@anonrig anonrig enabled auto-merge (squash) August 7, 2025 21:28
@anonrig anonrig disabled auto-merge August 7, 2025 21:34
@anonrig anonrig enabled auto-merge (squash) August 7, 2025 21:46
@npaun npaun disabled auto-merge August 7, 2025 23:41
@anonrig anonrig force-pushed the yagiz/add-http-socket branch from 168a3f7 to 8105d76 Compare August 8, 2025 18:04
@anonrig anonrig enabled auto-merge (squash) August 8, 2025 18:15
@anonrig anonrig merged commit 52c0434 into main Aug 8, 2025
32 of 33 checks passed
@anonrig anonrig deleted the yagiz/add-http-socket branch August 8, 2025 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants