X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fcharging-station%2Fui-server%2FUIWebSocketServer.ts;h=bf9eb8b3a66b753907d98a2df18120dcd25291ce;hb=fc28853f03af7666e2f55a244b787157fddadfbc;hp=c05ec6251278da2d3eed84711f975d7fb6e32ff1;hpb=2896e06dc8d72adf7150b23c941079f622f6f37c;p=e-mobility-charging-stations-simulator.git diff --git a/src/charging-station/ui-server/UIWebSocketServer.ts b/src/charging-station/ui-server/UIWebSocketServer.ts index c05ec625..bf9eb8b3 100644 --- a/src/charging-station/ui-server/UIWebSocketServer.ts +++ b/src/charging-station/ui-server/UIWebSocketServer.ts @@ -1,5 +1,5 @@ -import type { IncomingMessage } from 'http'; -import type internal from 'stream'; +import type { IncomingMessage } from 'node:http'; +import type { Duplex } from 'node:stream'; import { StatusCodes } from 'http-status-codes'; import WebSocket, { type RawData, WebSocketServer } from 'ws'; @@ -10,8 +10,7 @@ import { type UIServerConfiguration, WebSocketCloseEventStatusCode, } from '../../types'; -import { logger } from '../../utils/Logger'; -import { Utils } from '../../utils/Utils'; +import { Constants, Utils, logger } from '../../utils'; import { AbstractUIServer, UIServerUtils } from '../internal'; const moduleName = 'UIWebSocketServer'; @@ -49,12 +48,7 @@ export class UIWebSocketServer extends AbstractUIServer { } const [requestId] = request as ProtocolRequest; this.responseHandlers.set(requestId, ws); - this.uiServices - .get(version) - ?.requestHandler(request) - .catch(() => { - /* Error caught by AbstractUIService */ - }); + this.uiServices.get(version)?.requestHandler(request).catch(Constants.EMPTY_FUNCTION); }); ws.on('error', (error) => { logger.error(`${this.logPrefix(moduleName, 'start.ws.onerror')} WebSocket error:`, error); @@ -71,37 +65,34 @@ export class UIWebSocketServer extends AbstractUIServer { }); }); // eslint-disable-next-line @typescript-eslint/no-unused-vars - this.httpServer.on('connect', (req: IncomingMessage, socket: internal.Duplex, head: Buffer) => { + this.httpServer.on('connect', (req: IncomingMessage, socket: Duplex, head: Buffer) => { if (req.headers?.connection !== 'Upgrade' || req.headers?.upgrade !== 'websocket') { socket.write(`HTTP/1.1 ${StatusCodes.BAD_REQUEST} Bad Request\r\n\r\n`); socket.destroy(); } }); - this.httpServer.on( - 'upgrade', - (req: IncomingMessage, socket: internal.Duplex, head: Buffer): void => { - this.authenticate(req, (err) => { - if (err) { - socket.write(`HTTP/1.1 ${StatusCodes.UNAUTHORIZED} Unauthorized\r\n\r\n`); - socket.destroy(); - return; - } - try { - this.webSocketServer.handleUpgrade(req, socket, head, (ws: WebSocket) => { - this.webSocketServer.emit('connection', ws, req); - }); - } catch (error) { - logger.error( - `${this.logPrefix( - moduleName, - 'start.httpServer.on.upgrade' - )} Error at handling connection upgrade:`, - error - ); - } - }); - } - ); + this.httpServer.on('upgrade', (req: IncomingMessage, socket: Duplex, head: Buffer): void => { + this.authenticate(req, (err) => { + if (err) { + socket.write(`HTTP/1.1 ${StatusCodes.UNAUTHORIZED} Unauthorized\r\n\r\n`); + socket.destroy(); + return; + } + try { + this.webSocketServer.handleUpgrade(req, socket, head, (ws: WebSocket) => { + this.webSocketServer.emit('connection', ws, req); + }); + } catch (error) { + logger.error( + `${this.logPrefix( + moduleName, + 'start.httpServer.on.upgrade' + )} Error at handling connection upgrade:`, + error + ); + } + }); + }); this.startHttpServer(); }