Skip to content

v4.x Backport - test: don't connect to :: (use localhost instead) - #11740

Closed
gibfahn wants to merge 1 commit into
nodejs:v4.x-stagingfrom
gibfahn:listen-localhost-v4
Closed

v4.x Backport - test: don't connect to :: (use localhost instead)#11740
gibfahn wants to merge 1 commit into
nodejs:v4.x-stagingfrom
gibfahn:listen-localhost-v4

Conversation

@gibfahn

Copy link
Copy Markdown
Member

Backport of #10854

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

@nodejs-github-botnodejs-github-bot added test Issues and PRs related to the tests. v4.x labels Mar 8, 2017
If a test does http.listen(0) or net.listen(0),
http.listen(0).address().address returns '::'. Some machines will
resolve this to localhost, but not all. Every machine should have
localhost defined in /etc/hosts (or equivalent), so it should always
resolve.
Fixes: nodejs#7291
@gibfahn
gibfahnforce-pushed the listen-localhost-v4 branch from 6f25609 to ef8556eCompareMarch 8, 2017 07:41
@gibfahn

Copy link
Copy Markdown
MemberAuthor

@mscdexmscdex added the net Issues and PRs related to the net subsystem. label Mar 8, 2017
@MylesBorins

Copy link
Copy Markdown
Contributor

landed in abee209

@gibfahn
gibfahn deleted the listen-localhost-v4 branch March 8, 2017 10:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

netIssues and PRs related to the net subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gibfahn@MylesBorins@mscdex@nodejs-github-bot