Fall back to a random port if preferred port is privileged (#39)

sindresorhus · Dec 18, 2019 · 45a71bc · 45a71bc
1 parent 73e21d8
commit 45a71bc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -70,7 +70,7 @@ module.exports = async options => {
 			lockedPorts.young.add(availablePort);
 			return availablePort;
 		} catch (error) {
-			if (error.code !== 'EADDRINUSE' && !(error instanceof Locked)) {
+			if (!['EADDRINUSE', 'EACCES'].includes(error.code) && !(error instanceof Locked)) {
 				throw error;
 			}
 		}

diff --git a/test.js b/test.js
@@ -32,6 +32,15 @@ test('preferred port unavailable', async t => {
 	t.not(port, desiredPort);
 });
 
+test('preferred port privileged', async t => {
+	const desiredPort = 80;
+	const port = await getPort({host: '127.0.0.1', port: desiredPort});
+
+	t.is(typeof port, 'number');
+	t.true(port > 0);
+	t.not(port, desiredPort);
+});
+
 test('port can be bound to IPv4 host when promise resolves', async t => {
 	const port = await getPort({host: '0.0.0.0'});
 	t.is(typeof port, 'number');