From 70eb8e05c49e90bdaded5571010d8a4d51abb438 Mon Sep 17 00:00:00 2001 From: Matthew Allan Date: Mon, 2 Aug 2021 09:33:13 -0400 Subject: [PATCH] Only raise a warning, not an error for privileged ports On MacOS (and on Linux if using authbind) you don't actually need to use sudo to bind to ports < 1024, so node-foreman does not need to raise an error if that happens. Since we can't detect if this system will or will not require sudo we can just raise a warning. Signed-off-by: Matthew Allan --- lib/proc.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/proc.js b/lib/proc.js index 846887c..5b28aaf 100644 --- a/lib/proc.js +++ b/lib/proc.js @@ -107,8 +107,8 @@ function start(procs, requirements, envs, portarg, emitter){ var port = parseInt(portarg); if(port < 1024) { - return cons.Error('Only Proxies Can Bind to Privileged Ports - '+ - 'Try \'sudo nf start -x %s\'', port); + cons.Warn('May Not Be Able To Bind to Privileged Port - '+ + 'If Start Fails Try \'sudo nf start -x %s\'', port); }