Merge pull request #1841 from CodeNow/unsafe-url

Added http preamble to avoid unsafe urls
CodeNow · Oct 11, 2016 · 75c14ac · 75c14ac
2 parents daf24ed + b9c7fa5
commit 75c14ac
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/client/services/defaultContainerUrlService.js b/client/services/defaultContainerUrlService.js
@@ -9,11 +9,17 @@ function defaultContainerUrl(
   return function (instance) {
     var defaultPort = '';
     var ports = extractInstancePorts(instance);
+    var preamble = 'http://';
     if (ports.length) {
       if (!ports.includes('80')) {
-        defaultPort = ':' + ports[0];
+        if (ports.includes('443')) {
+          preamble = 'https://';
+        } else {
+          defaultPort = ':' + ports[0];
+        }
       }
     }
-    return instance.getContainerHostname() + defaultPort;
+
+    return preamble + instance.getContainerHostname() + defaultPort;
   };
 }