websocket: use Buffer.allocUnsafe (nodejs#1817)

Author: KhafraDev

lib/websocket/frame.js

@@ -27,9 +27,10 @@ class WebsocketFrameSend {
       payloadLength = 126
     }
 
-    // TODO: switch to Buffer.allocUnsafe
-    const buffer = Buffer.alloc(bodyLength + offset)
+    const buffer = Buffer.allocUnsafe(bodyLength + offset)
 
+    // Clear first 2 bytes, everything else is overwritten
+    buffer[0] = buffer[1] = 0
     buffer[0] |= 0x80 // FIN
     buffer[0] = (buffer[0] & 0xF0) + opcode // opcode
 

lib/websocket/receiver.js

@@ -198,6 +198,7 @@ class ByteParser extends Writable {
       const buffer = this.consume(8)
 
       // TODO: optimize this
+      // TODO: this likely doesn't work because it returns a bigint
       this.#info.payloadLength = buffer.readBigUint64BE(0)
       this.#state = parserStates.READ_DATA
     } else if (this.#state === parserStates.READ_DATA) {