From 3d3d6af457cc47200600ef53135ba2462b34ac4d Mon Sep 17 00:00:00 2001
From: Sam Wilkins <samwilkins333@gmail.com>
Date: Fri, 15 May 2020 21:44:06 -0700
Subject: server initialization and port resolution fixes

---
 src/server/server_Initialization.ts | 26 +++++++++++++-------------
 src/server/websocket.ts             |  7 ++++---
 2 files changed, 17 insertions(+), 16 deletions(-)

(limited to 'src')

diff --git a/src/server/server_Initialization.ts b/src/server/server_Initialization.ts
index 6866b01e3..f572ec906 100644
--- a/src/server/server_Initialization.ts
+++ b/src/server/server_Initialization.ts
@@ -19,10 +19,11 @@ import * as fs from 'fs';
 import * as request from 'request';
 import RouteSubscriber from './RouteSubscriber';
 import { publicDirectory } from '.';
-import { logPort, } from './ActionUtilities';
+import { logPort, pathFromRoot, } from './ActionUtilities';
 import { blue, yellow } from 'colors';
 import * as cors from "cors";
-var https = require('https');
+import { createServer, Server as SecureServer } from "https";
+import { Server } from "http";
 
 /* RouteSetter is a wrapper around the server that prevents the server
    from being exposed. */
@@ -49,28 +50,27 @@ export default async function InitializeServer(routeSetter: RouteSetter) {
     routeSetter(new RouteManager(app, isRelease));
     registerRelativePath(app);
 
-    const serverPort = isRelease ? Number(process.env.serverPort) : 1050;
+    const { serverPort } = process.env;
+    const resolved = isRelease && serverPort ? Number(serverPort) : 1050;
 
-    let server: any;
+    let server: Server | SecureServer;
     if (isRelease) {
-        server = https.createServer({
-            key: fs.readFileSync(`./${process.env.serverName}.key`),
-            cert: fs.readFileSync(`./${process.env.serverName}.cert`)
+        server = createServer({
+            key: fs.readFileSync(pathFromRoot(`./${process.env.serverName}.key`)),
+            cert: fs.readFileSync(pathFromRoot(`./${process.env.serverName}.crt`))
         }, app);
-        server.listen(serverPort, function () {
-            logPort("server", serverPort);
+        (server as SecureServer).listen(resolved, () => {
+            logPort("server", resolved);
             console.log();
-            // console.log('Example app listening on port 3000! Go to https://localhost:3000/')
         });
     } else {
-        server = app.listen(serverPort, () => {
-            logPort("server", serverPort);
+        server = app.listen(resolved, () => {
+            logPort("server", resolved);
             console.log();
         });
     }
 
     disconnect = async () => new Promise<Error>(resolve => server.close(resolve));
-
     return isRelease;
 }
 
diff --git a/src/server/websocket.ts b/src/server/websocket.ts
index 7278bdc32..9ab3d2611 100644
--- a/src/server/websocket.ts
+++ b/src/server/websocket.ts
@@ -122,9 +122,10 @@ export namespace WebSocket {
             };
         });
 
-        const socketPort = isRelease ? Number(process.env.socketPort) : 4321;
-        endpoint.listen(socketPort);
-        logPort("websocket", socketPort);
+        const { socketPort } = process.env;
+        const resolved = isRelease && socketPort ? Number(socketPort) : 4321;
+        endpoint.listen(resolved);
+        logPort("websocket", resolved);
     }
 
     function processGesturePoints(socket: Socket, content: GestureContent) {
-- 
cgit v1.2.3-70-g09d2