https: support agent construction without new

Fixes: nodejs#12918 PR-URL: nodejs#12927 Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
anchnk · May 19, 2017 · 47c4d96 · 47c4d96
1 parent b7cfca7
commit 47c4d96
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/lib/https.js b/lib/https.js
@@ -119,6 +119,9 @@ function createConnection(port, host, options) {
 
 
 function Agent(options) {
+  if (!(this instanceof Agent))
+    return new Agent(options);
+
   http.Agent.call(this, options);
   this.defaultPort = 443;
   this.protocol = 'https:';

diff --git a/test/parallel/test-https-agent-constructor.js b/test/parallel/test-https-agent-constructor.js
@@ -0,0 +1,7 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+const https = require('https');
+
+assert.doesNotThrow(() => { https.Agent(); });
+assert.ok(https.Agent() instanceof https.Agent);