Merge pull request #5 from arihantjn53/ACE_1059_remove_any_protocol

Remove any protocol part from --proxy-host

Author: francisf

config/constants.js

@@ -10,6 +10,7 @@ module.exports.PORTS = {
   MAX: 65535,
   MIN: 1
 };
+module.exports.PROTOCOL_REGEX = /(^\w+:|^)\/\//;
 
 module.exports.LOGS = Object.freeze({
   NETWORK: 'NetworkStats.log',

src/commandLine.js

@@ -119,8 +119,10 @@ var CommandLineManager = {
     index = argv.indexOf('--proxy-host');
     if (index !== -1) {
       if (CommandLineManager.validArgValue(argv[index + 1])) {
+        var host = argv[index + 1];
+        host = host.replace(constants.PROTOCOL_REGEX, '');
         RdGlobalConfig.proxy = RdGlobalConfig.proxy || {};
-        RdGlobalConfig.proxy.host = argv[index + 1];
+        RdGlobalConfig.proxy.host = host;
         argv.splice(index, 2);
       } else {
         invalidArgs.add('--proxy-host');

test/commandLine.test.js

@@ -38,6 +38,20 @@ describe('CommandLineManager', function () {
       expect(RdGlobalConfig.proxy.port).to.eql(9687);
     });
 
+    it('remove any protocol part from proxy-host', function () {
+      sinon.stub(console, 'log');
+      argv = argv.concat(['--proxy-host', 'http://host']);
+      CommandLineManager.processArgs(argv);
+      expect(RdGlobalConfig.proxy.host).to.eql('host');
+    });
+
+    it('remove any prefix slashes from proxy-host', function () {
+      argv = argv.concat(['--proxy-host', '//host']);
+      CommandLineManager.processArgs(argv);
+      console.log.restore();
+      expect(RdGlobalConfig.proxy.host).to.eql('host');
+    });
+
     it('proxy-port is set to the default value when its not in the expected range', function () {
       sinon.stub(console, 'log');
       argv = argv.concat(['--proxy-host', 'host', '--proxy-port', '99999']);