Immediately exit run command if server name collision detected

Fixes jeansaad#32
appjudo · Jun 24, 2022 · 050f3ba · 050f3ba
1 parent 0d7eba9
commit 050f3ba
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/cli/run.js b/src/cli/run.js
@@ -26,7 +26,7 @@ module.exports = {
       const serverAddress = `http://localhost:${port}`;
 
       process.env.PORT = port;
-      servers.add(serverAddress, opts);
+      if (!servers.add(serverAddress, opts)) process.exit(1);
 
       signals.forEach(signal => process.on(signal, cleanAndExit));
 

diff --git a/src/cli/servers.js b/src/cli/servers.js
@@ -110,7 +110,7 @@ function add(param, opts = {}) {
   fs.writeFileSync(file, data);
 
   // if we're mapping a domain to a URL there's no additional info to output
-  if (conf.target) return;
+  if (conf.target) return conf;
 
   // if we're mapping a domain to a local server add some info
   if (conf.out) {
@@ -123,6 +123,8 @@ function add(param, opts = {}) {
   if (!opts.p) {
     console.log("Port Random port (use '-p 1337' to set a fixed port)");
   }
+
+  return conf;
 }
 
 function rm(opts = {}) {