fix(https2): ipv6 addresses url

.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,6 +21,10 @@
   };
 }
 
+function normalizeIpv6Host(host) {
+  return net.isIP(host) === 6 ? `[${host}]` : host;
+}
+
 class Request extends Stream {
   constructor(protocol, options) {
     super();
@@ -48,11 +52,12 @@
 
     this._headers = {};
 
+    const normalizedHost = normalizeIpv6Host(host);
     const session = http2.connect(
-      `${protocol}//${host}:${port}`,
+      `${protocol}//${normalizedHost}:${port}`,
       sessionOptions
     );
-    this.setHeader('host', `${host}:${port}`);
+    this.setHeader('host', `${normalizedHost}:${port}`);
 
     session.on('error', (error) => this.emit('error', error));
 
@@ -73,7 +78,7 @@
     }
   }
 
   setNoDelay(bool) {
     // We can not use setNoDelay with HTTP/2.
     // Node 10 limits http2session.socket methods to ones safe to use with HTTP/2.
     // See also https://nodejs.org/api/http2.html#http2_http2session_socket
@@ -95,7 +100,7 @@
 
     const frame = this.session.request(headers);
 
     frame.once('response', (headers, flags) => {
       headers = this.mapToHttpHeader(headers);
       frame.headers = headers;
       frame.statusCode = headers[HTTP2_HEADER_STATUS];
@@ -179,7 +184,7 @@
     frame.end(data);
   }
 
   abort(data) {
     const frame = this.getFrame();
     frame.close(NGHTTP2_CANCEL);
     this.session.destroy();

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) => {