Union sock options

lib/Server.js

@@ -91,10 +91,14 @@ class Server {
     this.hot = this.options.hot || this.options.hotOnly;
     this.watchOptions = options.watchOptions || {};
 
+    if (!this.options.clientSocketOptions) {
+      this.options.clientSocketOptions = {};
+    }
+
     // Replace leading and trailing slashes to normalize path
     this.sockPath = `/${
-      this.options.sockPath
-        ? this.options.sockPath.replace(/^\/|\/$/g, '')
+      this.options.clientSocketOptions.path
+        ? this.options.clientSocketOptions.path.replace(/^\/|\/$/g, '')
         : 'sockjs-node'
     }`;
 

lib/options.json

@@ -39,6 +39,27 @@
     "clientLogLevel": {
       "enum": ["info", "warn", "error", "debug", "trace", "silent"]
     },
+    "clientSocketOptions": {
+      "type": "object",
+      "properties": {
+        "host": {
+          "type": "string"
+        },
+        "path": {
+          "type": "string"
+        },
+        "port": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "type": "number"
+            }
+          ]
+        }
+      }
+    },
     "compress": {
       "type": "boolean"
     },
@@ -297,25 +318,6 @@
     "setup": {
       "instanceof": "Function"
     },
-    "sockHost": {
-      "type": "string"
-    },
-    "sockPath": {
-      "type": "string"
-    },
-    "sockPort": {
-      "anyOf": [
-        {
-          "type": "number"
-        },
-        {
-          "type": "string"
-        },
-        {
-          "type": "null"
-        }
-      ]
-    },
     "socket": {
       "type": "string"
     },
@@ -403,6 +405,7 @@
       "ca": "should be {String|Buffer}",
       "cert": "should be {String|Buffer}",
       "clientLogLevel": "should be {String} and equal to one of the allowed values\n\n [ 'silent', 'info', 'debug', 'trace', 'error', 'warn' ]\n\n (https://webpack.js.org/configuration/dev-server/#devserverclientloglevel)",
+      "clientSocketOptions": "should be {Object} (https://webpack.js.org/configuration/dev-server/#devserverclientsocketoptions)",
       "compress": "should be {Boolean} (https://webpack.js.org/configuration/dev-server/#devservercompress)",
       "contentBase": "should be {Number|String|Array} (https://webpack.js.org/configuration/dev-server/#devservercontentbase)",
       "disableHostCheck": "should be {Boolean} (https://webpack.js.org/configuration/dev-server/#devserverdisablehostcheck)",
@@ -446,9 +449,6 @@
       "serveIndex": "should be {Boolean} (https://webpack.js.org/configuration/dev-server/#devserverserveindex)",
       "serverSideRender": "should be {Boolean} (https://github.com/webpack/webpack-dev-middleware#serversiderender)",
       "setup": "should be {Function} (https://webpack.js.org/configuration/dev-server/#devserversetup)",
-      "sockHost": "should be {String|Null} (https://webpack.js.org/configuration/dev-server/#devserversockhost)",
-      "sockPath": "should be {String} (https://webpack.js.org/configuration/dev-server/#devserversockpath)",
-      "sockPort": "should be {Number|String|Null} (https://webpack.js.org/configuration/dev-server/#devserversockport)",
       "socket": "should be {String} (https://webpack.js.org/configuration/dev-server/#devserversocket)",
       "staticOptions": "should be {Object} (https://webpack.js.org/configuration/dev-server/#devserverstaticoptions)",
       "stats": "should be {Object|Boolean} (https://webpack.js.org/configuration/dev-server/#devserverstats-)",

lib/utils/addEntries.js

@@ -27,14 +27,18 @@ function addEntries(config, options, server) {
       },
     };
 
+    if (!options.clientSocketOptions) {
+      options.clientSocketOptions = {};
+    }
+
     /** @type {string} */
     const domain = createDomain(options, app);
     /** @type {string} */
-    const sockHost = options.sockHost ? `&sockHost=${options.sockHost}` : '';
+    const sockHost = options.clientSocketOptions.host ? `&sockHost=${options.clientSocketOptions.host}` : '';
     /** @type {string} */
-    const sockPath = options.sockPath ? `&sockPath=${options.sockPath}` : '';
+    const sockPath = options.clientSocketOptions.path ? `&sockPath=${options.clientSocketOptions.path}` : '';
     /** @type {string} */
-    const sockPort = options.sockPort ? `&sockPort=${options.sockPort}` : '';
+    const sockPort = options.clientSocketOptions.port ? `&sockPort=${options.clientSocketOptions.port}` : '';
     /** @type {string} */
     const clientEntry = `${require.resolve(
       '../../client/'

lib/utils/createConfig.js

@@ -31,16 +31,23 @@ function createConfig(config, argv, { port }) {
     options.socket = argv.socket;
   }
 
+  if (
+    (argv.sockHost || argv.sockPath || argv.sockPort) &&
+    !options.clientSocketOptions
+  ) {
+    options.clientSocketOptions = {};
+  }
+
   if (argv.sockHost) {
-    options.sockHost = argv.sockHost;
+    options.clientSocketOptions.host = argv.sockHost;
   }
 
   if (argv.sockPath) {
-    options.sockPath = argv.sockPath;
+    options.clientSocketOptions.path = argv.sockPath;
   }
 
   if (argv.sockPort) {
-    options.sockPort = argv.sockPort;
+    options.clientSocketOptions.port = argv.sockPort;
   }
 
   if (argv.liveReload === false) {

test/e2e/ClientOptions.test.js

@@ -96,7 +96,9 @@ describe('Client complex inline script path', () => {
         poll: true,
       },
       public: 'myhost.test',
-      sockPath: '/foo/test/bar/',
+      clientSocketOptions: {
+        path: '/foo/test/bar/',
+      },
       quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
@@ -138,8 +140,10 @@ describe('Client complex inline script path with sockPort', () => {
       watchOptions: {
         poll: true,
       },
-      sockPath: '/foo/test/bar/',
-      sockPort: port3,
+      clientSocketOptions: {
+        path: '/foo/test/bar/',
+        port: port3,
+      },
       quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
@@ -185,7 +189,9 @@ describe('Client complex inline script path with sockPort, no sockPath', () => {
       watchOptions: {
         poll: true,
       },
-      sockPort: port3,
+      clientSocketOptions: {
+        port: port3,
+      },
       quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);
@@ -225,7 +231,9 @@ describe('Client complex inline script path with sockHost', () => {
       watchOptions: {
         poll: true,
       },
-      sockHost: 'myhost.test',
+      clientSocketOptions: {
+        host: 'myhost.test',
+      },
       quiet: true,
     };
     testServer.startAwaitingCompilation(config, options, done);

test/options.test.js

@@ -290,16 +290,8 @@ describe('options', () => {
         success: [''],
         failure: [false],
       },
-      sockHost: {
-        success: [''],
-        failure: [false],
-      },
-      sockPath: {
-        success: [''],
-        failure: [false],
-      },
-      sockPort: {
-        success: ['', 0, null],
+      clientSocketOptions: {
+        success: [{}],
         failure: [false],
       },
       staticOptions: {

test/server/sockPath-option.test.js

@@ -37,7 +37,9 @@ describe('sockPath options', () => {
       server = testServer.start(
         config,
         {
-          sockPath: '/foo/test/bar/',
+          clientSocketOptions: {
+            path: '/foo/test/bar/',
+          },
           port,
         },
         done

test/server/transportMode-option.test.js

@@ -195,7 +195,9 @@ describe('transportMode', () => {
             config,
             {
               port,
-              sockPath: '/foo/test/bar/',
+              clientSocketOptions: {
+                path: '/foo/test/bar/',
+              },
               transportMode: {
                 server: class MySockJSServer extends BaseServer {
                   constructor(serv) {
@@ -217,7 +219,7 @@ describe('transportMode', () => {
                       prefix: this.server.sockPath,
                     });
 
-                    sockPath = server.options.sockPath;
+                    sockPath = server.options.clientSocketOptions.path;
                   }
 
                   send(connection, message) {

test/server/utils/__snapshots__/createConfig.test.js.snap

@@ -1202,12 +1202,14 @@ Object {
 
 exports[`createConfig sockHost option 1`] = `
 Object {
+  "clientSocketOptions": Object {
+    "host": true,
+  },
   "hot": true,
   "hotOnly": false,
   "noInfo": true,
   "port": 8080,
   "publicPath": "/",
-  "sockHost": true,
   "stats": Object {
     "cached": false,
     "cachedAssets": false,
@@ -1217,12 +1219,14 @@ Object {
 
 exports[`createConfig sockPath option 1`] = `
 Object {
+  "clientSocketOptions": Object {
+    "path": "path",
+  },
   "hot": true,
   "hotOnly": false,
   "noInfo": true,
   "port": 8080,
   "publicPath": "/",
-  "sockPath": "path",
   "stats": Object {
     "cached": false,
     "cachedAssets": false,
@@ -1232,12 +1236,14 @@ Object {
 
 exports[`createConfig sockPort option 1`] = `
 Object {
+  "clientSocketOptions": Object {
+    "port": "port",
+  },
   "hot": true,
   "hotOnly": false,
   "noInfo": true,
   "port": 8080,
   "publicPath": "/",
-  "sockPort": "port",
   "stats": Object {
     "cached": false,
     "cachedAssets": false,