Scripts: coerce live reload port to integer (#29196)

* Scripts: coerce live reload port to integer * Scripts: simplify by removing defaultPort argument
WordPress · Feb 23, 2021 · e9ab152 · e9ab152
1 parent 2a55b96
commit e9ab152
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/packages/scripts/config/webpack.config.js b/packages/scripts/config/webpack.config.js
@@ -76,6 +76,12 @@ const getJsonpFunctionIdentifier = () => {
 	);
 };
 
+const getLiveReloadPort = ( inputPort ) => {
+	const parsedPort = parseInt( inputPort, 10 );
+
+	return Number.isInteger( parsedPort ) ? parsedPort : 35729;
+};
+
 const config = {
 	mode,
 	entry: {
@@ -221,7 +227,7 @@ const config = {
 		// works when running watch mode.
 		! isProduction &&
 			new LiveReloadPlugin( {
-				port: process.env.WP_LIVE_RELOAD_PORT || 35729,
+				port: getLiveReloadPort( process.env.WP_LIVE_RELOAD_PORT ),
 			} ),
 		// WP_NO_EXTERNALS global variable controls whether scripts' assets get
 		// generated, and the default externals set.