fix: add hostname and port to bonjour name to prevent name collisions

webpack · Oct 10, 2019 · 61b949e · 61b949e
1 parent 6d1f24f
commit 61b949e
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 2 deletions.
diff --git a/lib/utils/runBonjour.js b/lib/utils/runBonjour.js
@@ -2,9 +2,10 @@
 
 function runBonjour({ port }) {
   const bonjour = require('bonjour')();
+  const os = require('os');
 
   bonjour.publish({
-    name: 'Webpack Dev Server',
+    name: `Webpack Dev Server ${os.hostname()}:${port}`,
     port,
     type: 'http',
     subtypes: ['webpack'],

diff --git a/...ls/__snapshots__/ runBonjour.test.js.snap → ...ils/__snapshots__/runBonjour.test.js.snap b/...ls/__snapshots__/ runBonjour.test.js.snap → ...ils/__snapshots__/runBonjour.test.js.snap
@@ -2,7 +2,7 @@
 
 exports[`runBonjour should call bonjour.publish 1`] = `
 Object {
-  "name": "Webpack Dev Server",
+  "name": "Webpack Dev Server hostname:1111",
   "port": 1111,
   "subtypes": Array [
     "webpack",

diff --git a/test/server/utils/ runBonjour.test.js → test/server/utils/runBonjour.test.js b/test/server/utils/ runBonjour.test.js → test/server/utils/runBonjour.test.js
@@ -2,6 +2,12 @@
 
 const runBonjour = require('../../../lib/utils/runBonjour');
 
+jest.mock('os', () => {
+  return {
+    hostname: () => 'hostname',
+  };
+});
+
 describe('runBonjour', () => {
   let mock;
   let publish = jest.fn();
@@ -32,4 +38,16 @@ describe('runBonjour', () => {
 
     expect(publish.mock.calls[0][0]).toMatchSnapshot();
   });
+
+  it('should call bonjour.publish with different name for different ports', () => {
+    runBonjour({
+      port: 1111,
+    });
+
+    runBonjour({
+      port: 2222,
+    });
+
+    expect(publish.mock.calls[0][0].name).not.toEqual(publish.mock.calls[1][0].name);
+  });
 });