Added type and range validation for port (#52)

* Add port type and range validation to micro-dev. * Correct syntax
vercel · Jan 4, 2018 · 10026cb · timneutkens · Jan 4, 2018 · 10026cb
1 parent 756f52c
commit 10026cb
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/serve.js b/lib/serve.js
@@ -22,8 +22,13 @@ module.exports = async (file, flags, restarting) => {
     : log(getModule(file), flags.limit)
   const server = serve(module)
 
-  // `3000` is the default port
-  let port = ~~flags.port
+  const { isNaN } = Number
+  let port = Number(flags.port)
+
+  if (isNaN(port) || (!isNaN(port) && (port < 1 || port >= 2 ** 16))) {
+    console.error(`Port option must be a number. Supplied: ${flags.port}`)
+    process.exit(1)
+  }
 
   // Check if the specified port is already in use (if none
   // is specified, the default one will be checked)