Skip to content

doc: clarify http Agent keepSocketAlive behavior - #66252

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
efekrskl:doc/keepSocketAlive-default-case
Oct 8, 2026
Merged

nodejs-github-bot merged 1 commit into
nodejs:mainfrom
efekrskl:doc/keepSocketAlive-default-case

Conversation

@efekrskl

Copy link
Copy Markdown
Member

Updates the http.Agent#keepSocketAlive() documentation to reflect its current behavior. Removes the outdated implementation example. The implementation has drifted away from the example long time ago see

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/http
  • @nodejs/net

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to Node.js documentation. http Issues and PRs related to the http subsystem. labels Sep 23, 2026
@pimterry pimterry added the commit-queue PRs queued for automated landing through the Commit Queue. label Oct 8, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 855e3f7 into nodejs:main Oct 8, 2026
29 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 855e3f7

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Oct 8, 2026
richardlau pushed a commit that referenced this pull request Oct 12, 2026
Signed-off-by: Efe Karasakal <[email protected]>
PR-URL: #66252
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to Node.js documentation. http Issues and PRs related to the http subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants