Merge pull request #602 from netlify/fix/error-no-targetport

show useful error if targetPort is unknown
netlify · Dec 12, 2019 · 4e7d443 · 4e7d443
2 parents 8d7d16a + 49f347f
commit 4e7d443
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/commands/dev/index.js b/src/commands/dev/index.js
@@ -23,8 +23,8 @@ const find = require('find-process')
 const {
   NETLIFYDEV,
   NETLIFYDEVLOG,
-  NETLIFYDEVWARN
-  // NETLIFYDEVERR
+  NETLIFYDEVWARN,
+  NETLIFYDEVERR
 } = require('../../utils/logo')
 const boxen = require('boxen')
 const { createTunnel, connectTunnel } = require('../../utils/live-tunnel')
@@ -125,7 +125,14 @@ function initializeProxy(port) {
 }
 
 async function startProxy(settings, addonUrls) {
-  await waitPort({ port: settings.proxyPort })
+  try {
+    await waitPort({ port: settings.proxyPort })
+  } catch(err) {
+    console.error(NETLIFYDEVERR, `Netlify Dev doesn't know what port your site is running on.`)
+    console.error(NETLIFYDEVERR, `Please set --targetPort.`)
+    this.exit(1)
+  }
+
   if (settings.functionsPort) {
     await waitPort({ port: settings.functionsPort })
   }