Skip to content

doc: document net Socket server property - #64364

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
efekrskl:doc/socket-server-doc
Jul 13, 2026
Merged

doc: document net Socket server property#64364
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
efekrskl:doc/socket-server-doc

Conversation

@efekrskl

Copy link
Copy Markdown
Member

Closes#47148, based on the change proposed in #63458.

Original patch and investigation by @Herrtian

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/net

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. net Issues and PRs related to the net subsystem. labels Jul 8, 2026
@efekrskl

Copy link
Copy Markdown
MemberAuthor

@Herrtian I revived your patch from #63458 so it wouldn't get lost, and credited you in the PR body. If you want, I can also add you as a co-author later, but I wanted to ask for your explicit approval for that first because we need to sign it off.

@pimterrypimterry left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Not sure what went wrong with the original PR process but thanks for picking this up @efekrskl!

@Herrtian

Copy link
Copy Markdown
Contributor

@efekrskl Thanks for reviewing, I would appreciate if you add me as a co-author.

Based on nodejs#63458.
Co-authored-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
Signed-off-by: Efe Karasakal <hi@efe.dev>
Signed-off-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
@efekrskl
efekrsklforce-pushed the doc/socket-server-doc branch from 1a91367 to 2f9130eCompareJuly 8, 2026 21:16
@efekrskl

Copy link
Copy Markdown
MemberAuthor

@efekrskl Thanks for reviewing, I would appreciate if you add me as a co-author.

I added you as co-author, thanks a lot for your contribution!

@pimterrypimterry added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 13, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 13, 2026
@nodejs-github-bot
nodejs-github-bot merged commit e84b517 into nodejs:mainJul 13, 2026
19 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in e84b517

aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Based on #63458.
Co-authored-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
Signed-off-by: Efe Karasakal <hi@efe.dev>
Signed-off-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
PR-URL: #64364
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Based on #63458.
Co-authored-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
Signed-off-by: Efe Karasakal <hi@efe.dev>
Signed-off-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
PR-URL: #64364
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Based on #63458.
Co-authored-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
Signed-off-by: Efe Karasakal <hi@efe.dev>
Signed-off-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
PR-URL: #64364
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Based on #63458.
Co-authored-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
Signed-off-by: Efe Karasakal <hi@efe.dev>
Signed-off-by: Herrtian <70463940+Herrtian@users.noreply.github.com>
PR-URL: #64364
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.netIssues and PRs related to the net subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing reference (and types) for response.socket.server

5 participants

@efekrskl@nodejs-github-bot@Herrtian@lpinca@pimterry