allow form.submit with url string param to use https

vmosyaykin · vmosyaykin · commit adbfa7092e6b · 2016-09-20T18:36:39.000+03:00
diff --git a/lib/form_data.js b/lib/form_data.js
@@ -385,7 +385,8 @@ FormData.prototype.submit = function(params, cb) {
     options = populate({
       port: params.port,
       path: params.pathname,
-      host: params.hostname
+      host: params.hostname,
+      protocol: params.protocol
     }, defaults);
 
   // use custom params
diff --git a/test/integration/test-submit-url-parsing.js b/test/integration/test-submit-url-parsing.js
@@ -0,0 +1,27 @@
+var http = require('http');
+var https = require('https');
+var common = require('../common');
+var assert = common.assert;
+var FormData = require(common.dir.lib + '/form_data');
+
+/**
+ * Test url parsing during submission
+ */
+
+var req;
+var form = new FormData();
+form.append('field', 'value');
+
+// Basic parsing
+req = form.submit('http://localhost:' + common.port + '/path', function() {});
+assert.strictEqual(req.path, '/path');
+assert.ok(req.agent instanceof http.Agent, 'req.agent instanceof http.Agent');
+assert.strictEqual(req.getHeader('Host'), 'localhost:' + common.port);
+req.abort();
+
+// HTTPS protocol handling
+req = form.submit('https://localhost:' + common.port + '/path', function() {});
+assert.ok(req.agent instanceof https.Agent, 'req.agent instanceof https.Agent');
+req.abort();
+
+
