@fails-components/webtransport 1.6.8 → 1.6.10

package/lib/http2/parserbase.js

@@ -5,18 +5,6 @@ import { PriorityScheduler } from './priorityscheduler.js'
 const pid = typeof process !== 'undefined' ? process.pid : 0
 const log = logger(`webtransport:parserbase(${pid})`)
 
-/**
- * @param{Number|bigint} int
- * @returns {Number}
- */
-export function lengthVarInt(int) {
-  if (BigInt(int) < 64n) return 1
-  if (BigInt(int) < 16384n) return 2
-  if (BigInt(int) < 1073741824n) return 4
-  /* if (BigInt(int) < 4611686018427387904 ) */
-  return 8
-}
-
 export class ParserBase {
   static PADDING = 0x190b4d38
   static WT_RESET_STREAM = 0x190b4d39
@@ -66,7 +54,7 @@ export class ParserBase {
 
   /**
    * @abstract
-   * @param {Buffer|Uint8Array} data
+   * @param {Uint8Array[]} data
    */
   // eslint-disable-next-line no-unused-vars
   parseData(data) {
@@ -282,10 +270,17 @@ export class ParserBase {
    * @param {{code:  number, reason: string}} opts
    */
   onCloseWebTransportSession({ code, reason }) {
-    this.session.jsobj.onClose({
-      errorcode: code,
-      error: reason
-    })
+    if (
+      !(
+        this.session.jsobj.state === 'failed' ||
+        this.session.jsobj.state === 'closed'
+      )
+    ) {
+      this.session.jsobj.onClose({
+        errorcode: code,
+        error: reason
+      })
+    }
     this.closeHttp2Stream(code) // is this necessary
   }
 
@@ -315,7 +310,7 @@ export class ParserBase {
    */
   removeStream(streamid) {
     this.scheduler.Unregister(streamid)
-    //this.wtstreams.delete(streamid) // why does this create troubles
+    this.wtstreams.delete(streamid)
   }
 
   /**

package/lib/http2/parserbasehttp2.js

@@ -4,68 +4,6 @@ import { ParserBase } from './parserbase.js'
  * @typedef {import('node:http2').Http2Stream} Http2Stream
  */
 
-/**
- * @param{{offset: Number, buffer: Buffer, size: Number}} bs
- */
-
-export function readUint32(bs) {
-  if (bs.offset + 4 > bs.size) return undefined
-  const toret = bs.buffer.readUInt32BE(bs.offset)
-  bs.offset += 4
-  return toret
-}
-
-/**
- * @param{{offset: Number, buffer: Buffer, size: Number}} bs
- */
-export function readVarInt(bs) {
-  if (bs.offset + 1 > bs.size) return undefined
-  let val = BigInt(bs.buffer.readUInt8(bs.offset))
-  bs.offset++
-  const prefix = Number(val) >>> 6
-  const intlength = 1 << prefix
-
-  if (bs.offset + intlength - 1 > bs.size) {
-    return undefined
-  }
-  val = val & 0x3fn
-  for (let i = 0; i < intlength - 1; i++) {
-    val = (val << 8n) | BigInt(bs.buffer.readUInt8(bs.offset))
-    bs.offset++
-  }
-  return val
-}
-
-/**
- * @param{{offset: Number, buffer: Buffer, size: Number}} bs
- * @param{Number|bigint} int
- */
-export function writeVarInt(bs, int) {
-  let numbytes = 8n
-  let msb = 0xc0n
-  const bint = BigInt(int)
-  if (bint < 64) {
-    numbytes = 1n
-    msb = 0x0n
-  } else if (bint < 16384) {
-    numbytes = 2n
-    msb = 0x40n
-  } else if (bint < 1073741824) {
-    numbytes = 4n
-    msb = 0x80n
-  }
-  bs.buffer.writeUInt8(
-    Number(msb | ((bint >> ((numbytes - 1n) * 8n)) & 0xffn)),
-    bs.offset
-  )
-  bs.offset++
-
-  for (let i = numbytes - 2n; i >= 0; i--) {
-    bs.buffer.writeUInt8(Number((bint >> (i * 8n)) & 0xffn), bs.offset)
-    bs.offset++
-  }
-}
-
 export class ParserBaseHttp2 extends ParserBase {
   /**
    * @param {import('../types').ParserHttp2Init} stream
@@ -96,7 +34,7 @@ export class ParserBaseHttp2 extends ParserBase {
     this.stream.on('readable', () => {
       let data
       while ((data = this.stream.read()) !== null) {
-        this.parseData(data)
+        this.parseData([data])
       }
     })
 
@@ -110,10 +48,17 @@ export class ParserBaseHttp2 extends ParserBase {
        * @param {Error} error
        */ (error) => {
         // readable error
-        this.session.jsobj.onClose({
-          errorcode: this.stream.rstCode,
-          error: error.toString()
-        })
+        if (
+          !(
+            this.session.jsobj.state === 'failed' ||
+            this.session.jsobj.state === 'closed'
+          )
+        ) {
+          this.session.jsobj.onClose({
+            errorcode: this.stream.rstCode,
+            error: error.toString()
+          })
+        }
       }
     )
 

package/lib/http2/stream.js

@@ -92,17 +92,26 @@ export class Http2WebTransportStream {
 
   /**
    * @param {Object} obj
-   * @param {Uint8Array|undefined} obj.data
+   * @param {Uint8Array[]|undefined} obj.data
    * @param {Boolean} obj.fin
    */
   recvData({ data, fin }) {
-    this.incomdata.push({ data, fin })
-    if (data && data?.byteLength > 0) {
-      const checkstream = this.flowController.updateHighestReceivedOffset(
-        data?.byteLength
-      )
+    let datasize = 0
+    if (data) {
+      const newincomdata = data.map((el, ind, array) => ({
+        data: el,
+        fin: fin && ind + 1 === array.length
+      }))
+      datasize = data.reduce((acc, curVal) => acc + curVal.byteLength, 0)
+      this.incomdata.push(...newincomdata)
+    } else if (fin) {
+      this.incomdata.push({ data: undefined, fin })
+    }
+    if (datasize > 0) {
+      const checkstream =
+        this.flowController.updateHighestReceivedOffset(datasize)
       const checksession =
-        this.sessionFlowController.updateHighestReceivedOffset(data?.byteLength)
+        this.sessionFlowController.updateHighestReceivedOffset(datasize)
       if (checksession && checkstream) {
         // As the highest received offset has changed, check to see if this is a
         // violation of flow control.
@@ -242,13 +251,14 @@ export class Http2WebTransportStream {
   onClose() {
     // ok, inform our session id manager
     this.streamIdManager.onStreamClosed(this.streamid)
+    // also remove from capsule parser
+    this.capsuleParser.removeStream(this.streamid)
   }
 
   /**
    * @param {Number} code
    */
   stopSending(code) {
-    this.closeIncoming()
     this.capsuleParser.writeCapsule({
       type: ParserBase.WT_STOP_SENDING,
       headerVints: [this.streamid, code],
@@ -366,7 +376,7 @@ export class Http2WebTransportStream {
       }
     }
     if (finsend) {
-      this.capsuleParser.removeStream(this.streamid)
+      this.closeOutgoing()
       return
     }
 

package/lib/session.js

@@ -545,7 +545,7 @@ export class HttpWTSession {
   onClose(args) {
     delete this.objint // not valid any more
 
-    if (this.state !== 'connected') {
+    if (this.state !== 'connected' && this.state !== 'draining') {
       log.error(
         'session was closed before state was "connected" - it was "%s"',
         this.state

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fails-components/webtransport",
-  "version": "1.6.8",
+  "version": "1.6.10",
   "description": "A component to add webtransport support (server and client) to node.js using libquiche",
   "exports": {
     ".": {