refactor: https protocol ternary consistency

dherault · Aug 8, 2022 · 2b2a8fa · 2b2a8fa
1 parent 57f7d87
commit 2b2a8fa
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 8 deletions.
diff --git a/src/events/websocket/HttpServer.js b/src/events/websocket/HttpServer.js
@@ -50,8 +50,8 @@ export default class HttpServer {
     }
 
     log.notice(
-      `Offline [http for websocket] listening on http${
-        httpsProtocol ? 's' : ''
+      `Offline [http for websocket] listening on ${
+        httpsProtocol ? 'https' : 'http'
       }://${host}:${websocketPort}`,
     )
   }

diff --git a/src/events/websocket/WebSocketServer.js b/src/events/websocket/WebSocketServer.js
@@ -58,8 +58,8 @@ export default class WebSocketServer {
     const { host, httpsProtocol, websocketPort } = this.#options
 
     log.notice(
-      `Offline [websocket] listening on ws${
-        httpsProtocol ? 's' : ''
+      `Offline [websocket] listening on ${
+        httpsProtocol ? 'wss' : 'ws'
       }://${host}:${websocketPort}`,
     )
   }

diff --git a/src/events/websocket/lambda-events/WebSocketConnectEvent.js b/src/events/websocket/lambda-events/WebSocketConnectEvent.js
@@ -37,7 +37,7 @@ export default class WebSocketConnectEvent {
     //   'X-Amzn-Trace-Id': `Root=${createUniqueId()}`,
     //   'X-Forwarded-For': '127.0.0.1',
     //   'X-Forwarded-Port': String(this.#websocketPort),
-    //   'X-Forwarded-Proto': `http${this.#httpsProtocol ? 's' : ''}`,
+    //   'X-Forwarded-Proto': ${httpsProtocol ? 'https' : 'http'},
     // }
 
     const headers = parseHeaders(this.#rawHeaders)

diff --git a/src/lambda/HttpServer.js b/src/lambda/HttpServer.js
@@ -44,13 +44,15 @@ export default class HttpServer {
     }
 
     log.notice(
-      `Offline [http for lambda] listening on http${
-        httpsProtocol ? 's' : ''
+      `Offline [http for lambda] listening on ${
+        httpsProtocol ? 'https' : 'http'
       }://${host}:${lambdaPort}`,
     )
 
     // Print all the invocation routes to debug
-    const basePath = `http${httpsProtocol ? 's' : ''}://${host}:${lambdaPort}`
+    const basePath = `${
+      httpsProtocol ? 'https' : 'http'
+    }://${host}:${lambdaPort}`
     const funcNamePairs = this.#lambda.listFunctionNamePairs()
 
     log.notice(