feat: Added --localOnly flag

Author: shakyShane

lib/connect-utils.js

@@ -28,8 +28,17 @@ var connectUtils = {
          */
         var scriptSrc = (function () {
 
+            if (options.get("localOnly")) {
+                return [
+                    options.get("scheme"),
+                    "://localhost:",
+                    options.get("port"),
+                    scriptPath
+                ].join("");
+            }
+
             /**
-             * First, was 'scriptPath' set? if so the user wanted full control over the
+             * First, was "scriptPath" set? if so the user wanted full control over the
              * script tag output
              *
              */
@@ -38,7 +47,7 @@ var connectUtils = {
             }
 
             /**
-             * Next, if 'script.domain' was given, allow that + the path to the JS file
+             * Next, if "script.domain" was given, allow that + the path to the JS file
              * eg:
              *  script.domain=localhost:3000
              * -> localhost:3000/browser-sync/browser-sync-client.js
@@ -76,7 +85,7 @@ var connectUtils = {
          * Decide which template shall be used to generate the script tags
          */
         var template = (function () {
-            if (scriptDomain) {
+            if (scriptDomain || options.get("localOnly")) {
                 return config.templates.scriptTagSimple;
             }
             return config.templates.scriptTag;
@@ -167,6 +176,14 @@ var connectUtils = {
          * Ensure socket.domain is always a string (for noop replacements later)
          */
         socketOpts.domain = (function () {
+            if (options.get("localOnly")) {
+                string = withDomain;
+                return [
+                    options.get("scheme"),
+                    "://localhost:",
+                    options.get("port")
+                ].join("");
+            }
             if (socketOpts.domain) {
                 string = withDomain;
                 /**

lib/default-config.js

@@ -383,6 +383,15 @@ module.exports = {
      */
     host: null,
 
+    /**
+     * Support environments where dynamic hostnames are not required
+     * (ie: electron)
+     * @property localOnly
+     * @type Boolean
+     * @default false
+     */
+    localOnly: false,
+
     /**
      * @property codeSync
      * @type Boolean

test/specs/utils/utils.connect.js

@@ -6,10 +6,6 @@ var req    = require("supertest");
 var merge  = require("../../../lib/cli/cli-options").merge;
 var assert = require("chai").assert;
 
-// server,proxy: ['' + location.host + '/browser-sync']
-// snippet:      ['http://' + location.hostname + ':3000/browser-sync']
-// domain:       ['<domain>/browser-sync']
-
 describe("Connection snippetUtils", function () {
     var options;
     beforeEach(function () {
@@ -173,6 +169,29 @@ describe("Connection snippetUtils", function () {
                 });
         });
     });
+    it("E2E Should allow setting of socket.domain + script.domain as strings when using --localOnly flag", function (done) {
+        bs.reset();
+        bs.create().init({
+            ui: false,
+            online: false,
+            logLevel: "silent",
+            localOnly: true
+        }, function (err, bs) {
+            var port = bs.options.get("port");
+            assert.include(bs.options.get("snippet"), "<script async id=\"__bs_script__\" src=\"http://localhost:" + port + "/browser-sync");
+
+            var expected = "___browserSync___.io('http://localhost:" + port + "/browser-sync'";
+
+            req(bs.server)
+                .get(bs.options.getIn(["scriptPaths", "path"]))
+                .expect(200)
+                .end(function (err, res) {
+                    assert.include(res.text, expected, "Socket domain updated in response");
+                    bs.cleanup();
+                    done();
+                });
+        });
+    });
     it("E2E Should allow setting of script.domain as functions", function (done) {
         bs.reset();
         bs.create().init({