Fix adding options.host to defaultHosts list

MasterOdin · web-flow · commit 5f12a95a77c4 · 2023-05-26T12:20:46.000-04:00
diff --git a/lib/portfinder.js b/lib/portfinder.js
@@ -141,10 +141,8 @@ exports.getPort = function (options, callback) {
     }
   }
 
-  if (options.host) {
-    if (exports._defaultHosts.indexOf(options.host) !== -1) {
-      exports._defaultHosts.push(options.host)
-    }
+  if (options.host && exports._defaultHosts.indexOf(options.host) === -1) {
+    exports._defaultHosts.push(options.host)
   }
 
   var openPorts = [], currentHost;

Original file line number	Diff line number	Diff line change
`@@ -141,10 +141,8 @@ exports.getPort = function (options, callback) {`
`141`	`141`	`}`
`142`	`142`	`}`
`143`	`143`
`144`		`- if (options.host) {`
`145`		`- if (exports._defaultHosts.indexOf(options.host) !== -1) {`
`146`		`- exports._defaultHosts.push(options.host)`
`147`		`- }`
	`144`	`+ if (options.host && exports._defaultHosts.indexOf(options.host) === -1) {`
	`145`	`+ exports._defaultHosts.push(options.host)`
`148`	`146`	`}`
`149`	`147`
`150`	`148`	`var openPorts = [], currentHost;`