fix: check if websocket is open before writing (#22)

itsfuad · darrachequesne · commit d08f8f2152e0 · 2025-01-09T08:26:41.000+01:00
Related: #23
diff --git a/packages/engine.io/lib/transports/websocket.ts b/packages/engine.io/lib/transports/websocket.ts
@@ -16,7 +16,7 @@ export class WS extends Transport {
   public send(packets: Packet[]) {
     for (const packet of packets) {
       Parser.encodePacket(packet, true, (data: RawData) => {
-        if (this.writable) {
+        if (this.writable && this.socket?.readyState === WebSocket.OPEN) {
           this.socket?.send(data);
         }
       });

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ export class WS extends Transport {`
`16`	`16`	`public send(packets: Packet[]) {`
`17`	`17`	`for (const packet of packets) {`
`18`	`18`	`Parser.encodePacket(packet, true, (data: RawData) => {`
`19`		`- if (this.writable) {`
	`19`	`+ if (this.writable && this.socket?.readyState === WebSocket.OPEN) {`
`20`	`20`	`this.socket?.send(data);`
`21`	`21`	`}`
`22`	`22`	`});`