diff options
author | Andreas Madsen <amwebdk@gmail.com> | 2018-01-14 13:09:24 +0300 |
---|---|---|
committer | Evan Lucas <evanlucas@me.com> | 2018-01-31 22:51:04 +0300 |
commit | e9397d67a3fb6abfe59dedf313ad58859fc1740b (patch) | |
tree | 9c24efd97be10bd6624714435acf24defee5d54e | |
parent | ac2f98d6a60770df4b085e39f780c3c139f21fec (diff) |
async_hooks,test: only use IPv6 in http test
If IPv6 is not supported on a machine, the IPv6 handle will first be
created, this will then fail and default to an IPv4 handle. This causes
the graph to change, as there now is an extra handle.
Backport-PR-URL: https://github.com/nodejs/node/pull/18474
PR-URL: https://github.com/nodejs/node/pull/18143
Fixes: https://github.com/nodejs/node/issues/18003
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
-rw-r--r-- | test/async-hooks/test-graph.http.js | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/test/async-hooks/test-graph.http.js b/test/async-hooks/test-graph.http.js index 5c0c99f408c..eea72ca3bac 100644 --- a/test/async-hooks/test-graph.http.js +++ b/test/async-hooks/test-graph.http.js @@ -1,6 +1,9 @@ 'use strict'; const common = require('../common'); +if (!common.hasIPv6) + common.skip('IPv6 support required'); + const initHooks = require('./init-hooks'); const verifyGraph = require('./verify-graph'); const http = require('http'); @@ -13,7 +16,11 @@ const server = http.createServer(common.mustCall(function(req, res) { this.close(common.mustCall()); })); server.listen(0, common.mustCall(function() { - http.get(`http://127.0.0.1:${server.address().port}`, common.mustCall()); + http.get({ + host: '::1', + family: 6, + port: server.address().port + }, common.mustCall()); })); process.on('exit', function() { |