Merge pull request #148 from MasterOdin/patch-1

eriktrom · web-flow · commit c87abaff3643 · 2025-02-26T13:42:24.000-08:00
Fix adding options.host to defaultHosts list
diff --git a/lib/portfinder.js b/lib/portfinder.js
@@ -143,10 +143,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
`@@ -143,10 +143,8 @@ exports.getPort = function (options, callback) {`
`143`	`143`	`}`
`144`	`144`	`}`
`145`	`145`
`146`		`- if (options.host) {`
`147`		`- if (exports._defaultHosts.indexOf(options.host) !== -1) {`
`148`		`- exports._defaultHosts.push(options.host)`
`149`		`- }`
	`146`	`+ if (options.host && exports._defaultHosts.indexOf(options.host) === -1) {`
	`147`	`+ exports._defaultHosts.push(options.host)`
`150`	`148`	`}`
`151`	`149`
`152`	`150`	`var openPorts = [], currentHost;`