allow form.submit with url string param to use https

vmosyaykin · vmosyaykin · commit 4043fbb7e988 · 2016-08-29T16:28:22.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,25 @@
+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(err, res) {});
+assert.strictEqual(req.path, '/path');
+assert.strictEqual(req.agent.protocol, 'http:');
+assert.strictEqual(req.getHeader('Host'), 'localhost:' + common.port);
+req.abort();
+
+// HTTPS protocol handling
+req = form.submit('https://localhost:' + common.port + '/path', function(err, res) {});
+assert.strictEqual(req.agent.protocol, 'https:');
+req.abort();
+
+
