fix(https2): ipv6 addresses url

fix

oops

Author: perrin4869

.dist.eslintrc

@@ -17,6 +17,7 @@
     "no-undef": "off",
     "no-unused-vars": "off",
     "no-useless-escape": "off",
+    "no-obj-calls": "off",
     "no-cond-assign": "off",
     "no-redeclare": "off",
     "node/no-exports-assign": "off",

src/node/http2wrapper.js

@@ -21,13 +21,17 @@ function setProtocol(protocol) {
   };
 }
 
+function normalizeIpv6Host(host) {
+  return net.isIP(host) === 6 ? `[${host}]` : host;
+}
+
 class Request extends Stream {
   constructor(protocol, options) {
     super();
     const defaultPort = protocol === 'https:' ? 443 : 80;
     const defaultHost = 'localhost';
     const port = options.port || defaultPort;
-    const host = options.host || defaultHost;
+    const host = normalizeIpv6Host(options.host || defaultHost);
 
     delete options.port;
     delete options.host;

test/node/basic.js

@@ -134,23 +134,21 @@ describe('[node] request', () => {
     });
   });
 
-  if (doesntWorkInHttp2) {
-    describe('ipv6 address', () => {
-      it('should successfully query an ipv6 address', (done) => {
-        request.get(`http://[::]:${process.env.ZUUL_PORT}/url?a=(b%29`).end((error, res) => {
-          assert.equal('/url?a=(b%29', res.text);
-          done();
-        });
+  describe('ipv6 address', () => {
+    it('should successfully query an ipv6 address', (done) => {
+      request.get(`http://[::]:${process.env.ZUUL_PORT}/url?a=(b%29`).end((error, res) => {
+        assert.equal('/url?a=(b%29', res.text);
+        done();
       });
+    });
 
-      it('should successfully query an ipv6 address', (done) => {
-        request.get(`http://[::1]:${process.env.ZUUL_PORT}/url?a=(b%29`).end((error, res) => {
-          assert.equal('/url?a=(b%29', res.text);
-          done();
-        });
+    it('should successfully query an ipv6 address', (done) => {
+      request.get(`http://[::1]:${process.env.ZUUL_PORT}/url?a=(b%29`).end((error, res) => {
+        assert.equal('/url?a=(b%29', res.text);
+        done();
       });
     });
-  }
+  });
 
   describe('.buffer()', () => {
     it('should enable buffering', (done) => {