Skip to content

Commit 78b6b3a

Browse files
committed
Add check for missing reconnection token
This means something is misconfigured.
1 parent c4fd725 commit 78b6b3a

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/server.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ export abstract class Server {
266266
await this.preHandleWebSocket(request, socket);
267267
} catch (error) {
268268
socket.destroy();
269-
console.error(error);
269+
console.error(error.message);
270270
}
271271
}
272272

@@ -421,8 +421,11 @@ export class MainServer extends Server {
421421
}
422422

423423
protected async handleWebSocket(socket: net.Socket, parsedUrl: url.UrlWithParsedQuery): Promise<void> {
424+
if (!parsedUrl.query.reconnectionToken) {
425+
throw new Error("Reconnection token is missing from query parameters");
426+
}
424427
const protocol = new Protocol(await this.createProxy(socket), {
425-
reconnectionToken: <string>parsedUrl.query.reconnectionToken || "",
428+
reconnectionToken: <string>parsedUrl.query.reconnectionToken,
426429
reconnection: parsedUrl.query.reconnection === "true",
427430
skipWebSocketFrames: parsedUrl.query.skipWebSocketFrames === "true",
428431
});

0 commit comments

Comments
 (0)