https: support agent construction without new

Fixes: #12918 PR-URL: #12927 Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
nodejs · May 11, 2017 · 9ce2271 · 9ce2271
1 parent abfd4bf
commit 9ce2271
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);