FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

net: support DNS hints in createConnection() by cjihrig · Pull Request #6061 · nodejs/node · GitHub

/ node Public

net: support DNS hints in createConnection() - #6061

Closed
cjihrig wants to merge 1 commit into
nodejs:v5.xfrom
cjihrig:v5.x
Closed

net: support DNS hints in createConnection()#6061
cjihrig wants to merge 1 commit into
nodejs:v5.xfrom
cjihrig:v5.x

Conversation

cjihrig commented Apr 5, 2016

Copy link
Copy Markdown
Contributor

Backport of #6000 to v5.x branch.

R= @thealphanerd

This commit adds support for passing DNS lookup hints to
createConnection().

PR-URL: nodejs#6000
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>

Conflicts:
	test/parallel/test-net-create-connection.js
mscdex added dns Issues and PRs related to the dns subsystem. net Issues and PRs related to the net subsystem. v5.x labels Apr 5, 2016
MylesBorins assigned GlenTiki and MylesBorins and unassigned GlenTiki Apr 5, 2016

jasnell commented Apr 7, 2016

Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

landed in v5.x as 6d9c0c9

MylesBorins removed their assignment Dec 27, 2016
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dns Issues and PRs related to the dns subsystem. net Issues and PRs related to the net subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL