Fixed bug in https handling (#568)

The default `return startServers()` needs to be wrapped in an `else` block because the first `return` statement was done in a callback (inside `pem.createCertificate`).
redfin · Aug 16, 2016 · 6adf661 · 6adf661
1 parent f69b753
commit 6adf661
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/react-server-cli/src/commands/start.js b/packages/react-server-cli/src/commands/start.js
@@ -66,10 +66,10 @@ export default function start(options){
 		// in this case, we assume that httpOptions is an object that can be passed
 		// in to https.createServer as options.
 		return startServers(httpsOptions);
+	} else {
+		// Default.  Use http.
+		return startServers();
 	}
-
-	// Default.  Use http.
-	return startServers();
 }
 
 // given the server routes file and a port, start a react-server HTML server at