fix: use location.port when location.hostname is used to infer HMR socket URL (#1664)

Author: rlamana

client-src/default/index.js

@@ -181,6 +181,7 @@ const onSocketMsg = {
 
 let hostname = urlParts.hostname;
 let protocol = urlParts.protocol;
+let port = urlParts.port;
 
 // check ipv4 and ipv6 `all hostname`
 if (hostname === '0.0.0.0' || hostname === '::') {
@@ -190,6 +191,7 @@ if (hostname === '0.0.0.0' || hostname === '::') {
   // eslint-disable-next-line no-bitwise
   if (self.location.hostname && !!~self.location.protocol.indexOf('http')) {
     hostname = self.location.hostname;
+    port = self.location.port;
   }
 }
 
@@ -208,7 +210,7 @@ const socketUrl = url.format({
   protocol,
   auth: urlParts.auth,
   hostname,
-  port: urlParts.port,
+  port,
   // If sockPath is provided it'll be passed in via the __resourceQuery as a
   // query param so it has to be parsed out of the querystring in order for the
   // client to open the socket to the correct location.

package-lock.json

@@ -82,6 +82,7 @@
     "marked": "^0.6.1",
     "nyc": "^13.3.0",
     "prettier": "^1.16.3",
+    "puppeteer": "^1.12.2",
     "rimraf": "^2.6.2",
     "standard-version": "^5.0.0",
     "style-loader": "^0.23.1",