Skip to content

Commit 8b87e97

Browse files
committed
Only raise a warning for privileged port binds
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 <[email protected]>
1 parent 782cf09 commit 8b87e97

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/proc.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,8 @@ function start(procs, requirements, envs, portarg, emitter){
107107
var port = parseInt(portarg);
108108

109109
if(port < 1024) {
110-
return cons.Error('Only Proxies Can Bind to Privileged Ports - '+
111-
'Try \'sudo nf start -x %s\'', port);
110+
cons.Warn('May Not Be Able To Bind to Privileged Port - '+
111+
'If Start Fails Try \'sudo nf start -x %s\'', port);
112112
}
113113

114114

0 commit comments

Comments
 (0)