@fails-components/webtransport 1.0.11 → 1.1.2

package/lib/http2/node/server.js

@@ -97,6 +97,7 @@ export class Http2WebTransportServer {
       // ok we got a session and want to measure RTT
       let pingsender = setInterval(() => {
         if (!session.closed)
+          // eslint-disable-next-line no-unused-vars
           session.ping((err, duration, payload) => {
             if (!err) {
               rtt = adjust * duration + (1 - adjust) * rtt
@@ -127,20 +128,31 @@ export class Http2WebTransportServer {
       'upgrade',
       (request, stream /* actually a socket */, head) => {
         let path = request.url
-        const header = request.headers
         if (!path) {
           stream.destroy()
           return
         }
+        while (path.length > 1 && path[0] === '/' && path[1] === '/') {
+          path = path?.slice(1)
+        }
+        const header = { ...request.headers, ':path': path }
         const websocketProt = this.checkProtocolHeader(header)
         if (!websocketProt) {
           stream.destroy()
           return
         }
-        while (path.length > 1 && path[0] === '/' && path[1] === '/') {
-          path = path?.slice(1)
-        }
-        if (this.paths[path]) {
+        if (this.hasrequesthandler) {
+          stream.head = head
+
+          const retObj = {
+            header,
+            session: stream,
+            protocol: 'websocketoverhttp1',
+            head,
+            transportPrivate: { websocketProt }
+          }
+          this.jsobj.onSessionRequest(retObj)
+        } else if (this.paths[path]) {
           this.sendHttp1Headers({ stream, header, protocol: websocketProt })
             .then(() => {
               const retObj = {
@@ -189,16 +201,6 @@ export class Http2WebTransportServer {
               log('Problem sendHttp1Header', error)
               stream.destroy()
             })
-        } else if (this.hasrequesthandler) {
-          stream.head = head
-
-          const retObj = {
-            header,
-            session: stream,
-            protocol: 'websocketoverhttp1',
-            head
-          }
-          this.jsobj.onSessionRequest(retObj)
         } else {
           stream.destroy()
         }
@@ -245,7 +247,16 @@ export class Http2WebTransportServer {
       while (path.length > 1 && path[0] === '/' && path[1] === '/') {
         path = path?.slice(1)
       }
-      if (this.paths[path]) {
+      header[':path'] = path // also adapt it for the middleware
+      if (this.hasrequesthandler) {
+        const retObj = {
+          header,
+          session: stream,
+          protocol: websocketProt ? 'websocket' : 'capsule',
+          transportPrivate: { websocketProt }
+        }
+        this.jsobj.onSessionRequest(retObj)
+      } else if (this.paths[path]) {
         const {
           0x2b65: remoteBidirectionalStreams = undefined,
           0x2b64: remoteUnidirectionalStreams = undefined,
@@ -327,19 +338,6 @@ export class Http2WebTransportServer {
         if (websocketProt) resp['sec-websocket-protocol'] = websocketProt
         stream.respond(resp)
         this.jsobj.onHttpWTSessionVisitor(retObj)
-      } else if (this.hasrequesthandler) {
-        const retObj = {
-          header,
-          session: stream,
-          protocol: websocketProt ? 'websocket' : 'capsule'
-        }
-        const resp = {
-          ':status': '200'
-        }
-        // @ts-ignore
-        if (websocketProt) resp['sec-websocket-protocol'] = websocketProt
-        stream.respond(resp)
-        this.jsobj.onSessionRequest(retObj)
       } else {
         stream.respond({
           ':status': '404'
@@ -381,10 +379,11 @@ export class Http2WebTransportServer {
   }
 
   /**
-   * @param {string} cert
-   * @param {string} privKey
+   * @param {string|string[]} cert
+   * @param {string|string[]} privKey
    * @param {boolean} http2only
    * */
+  // eslint-disable-next-line no-unused-vars
   updateCert(cert, privKey, http2only) {
     this.serverInt.setSecureContext({ key: privKey, cert })
   }
@@ -437,17 +436,35 @@ export class Http2WebTransportServer {
   /**
    * @param {import('../../types.js').NativeFinishSessionRequest} args
    */
-  finishSessionRequest({ header, session: stream, status, protocol, head }) {
+  finishSessionRequest({
+    header,
+    userData,
+    session: stream,
+    status,
+    protocol,
+    head,
+    path,
+    transportPrivate
+  }) {
     if (status !== 200) {
       if (protocol === 'websocketoverhttp1') {
         stream.destroy()
       } else {
-        stream.close(constants.HTTP_STATUS_NOT_FOUND)
+        stream.respond({
+          ':status': status.toString()
+        })
+        stream.end()
+        stream.close()
       }
     } else {
       if (protocol === 'websocketoverhttp1') {
-        // @ts-ignore
-        this.sendHttp1Headers({ stream, header })
+        this.sendHttp1Headers({
+          // @ts-ignore
+          stream,
+          header,
+          // @ts-ignore
+          protocol: transportPrivate.websocketProt
+        })
           .then(() => {
             const retObj = {
               session: new Http2WebTransportSession({
@@ -484,8 +501,9 @@ export class Http2WebTransportServer {
                   this.sessionShouldAutoTuneReceiveWindow,
                 receiveWindowSizeLimit: this.sessionFlowControlWindowSizeLimit
               }),
-              path: header[':path'],
-              header
+              path,
+              header,
+              userData
             }
             // @ts-ignore
             this.jsobj.onHttpWTSessionVisitor(retObj)
@@ -494,6 +512,14 @@ export class Http2WebTransportServer {
             log('sendHttp1Headers error', error)
           })
       } else {
+        const resp = {
+          ':status': '200'
+        }
+        // @ts-ignore
+        if (transportPrivate?.websocketProt)
+          // @ts-ignore
+          resp['sec-websocket-protocol'] = transportPrivate.websocketProt
+        stream.respond(resp)
         const {
           0x2b65: remoteBidirectionalStreams = undefined,
           0x2b64: remoteUnidirectionalStreams = undefined,
@@ -566,8 +592,9 @@ export class Http2WebTransportServer {
               this.sessionShouldAutoTuneReceiveWindow,
             receiveWindowSizeLimit: this.sessionFlowControlWindowSizeLimit
           }),
-          path: header[':path'],
-          header
+          path,
+          header,
+          userData
         }
         // @ts-ignore
         this.jsobj.onHttpWTSessionVisitor(retObj)

package/lib/http2/node/websocketparser.js

@@ -863,6 +863,7 @@ export class WebSocketParser extends ParserBaseHttp2 {
   /**
    * @param {number} code
    */
+  // eslint-disable-next-line no-unused-vars
   closeHttp2Stream(code) {
     const stream = this.stream
     if (stream.close) {

package/lib/http2/parserbase.js

@@ -65,6 +65,7 @@ export class ParserBase {
    * @abstract
    * @param {Buffer|Uint8Array} data
    */
+  // eslint-disable-next-line no-unused-vars
   parseData(data) {
     throw new Error('Implement parseData in derived Class')
   }
@@ -73,6 +74,7 @@ export class ParserBase {
    * @abstract
    * @param{{type: Number, headerVints: Array<Number|bigint>, payload: Uint8Array|undefined, end?: () => void}} bs
    */
+  // eslint-disable-next-line no-unused-vars
   writeCapsule({ type, headerVints, payload, end }) {
     throw new Error('Implement writeCapsule in derived Class')
   }
@@ -252,6 +254,7 @@ export class ParserBase {
   /**
    * @param {number} code
    */
+  // eslint-disable-next-line no-unused-vars
   closeHttp2Stream(code) {
     throw new Error('Implement closeHttp2Stream in derived Class')
   }

package/lib/http2/session.js

@@ -136,6 +136,7 @@ export class Http2WebTransportSession {
   /**
    * @param {{sendGroup:  WebTransportSendGroup|null, sendOrder: number, waitUntilAvailable: boolean}} opts
    */
+  // eslint-disable-next-line no-unused-vars
   orderUnidiStream({ sendGroup, sendOrder, waitUntilAvailable }) {
     const canopen = this.streamIdMngrUni.canOpenNextOutgoingStream()
     const maxset = this.streamIdMngrUni.isMaxStreamSet() // we block if the maxsetting did not arrive
@@ -167,6 +168,7 @@ export class Http2WebTransportSession {
   /**
    * @param {{sendGroup:  WebTransportSendGroup|null, sendOrder: number, waitUntilAvailable: boolean}} opts
    */
+  // eslint-disable-next-line no-unused-vars
   orderBidiStream({ sendGroup, sendOrder, waitUntilAvailable }) {
     const canopen = this.streamIdMngrBi.canOpenNextOutgoingStream()
     const maxset = this.streamIdMngrBi.isMaxStreamSet() // we block if the maxsetting did not arrive
@@ -265,6 +267,7 @@ export class Http2WebTransportSession {
     } else if (this.ws) {
       // we are at the Browser, so we use the connection rtt?
       // @ts-ignore
+      // eslint-disable-next-line no-undef
       return navigator?.connection?.rtt || 26
     }
   }

package/lib/http2/stream.js

@@ -173,7 +173,7 @@ export class Http2WebTransportStream {
               cur.data.byteOffset + len,
               cur.data.byteLength - len
             ),
-            fin: false || cur.fin
+            fin: cur.fin
           })
           buffer.fin = false // next round
         } else {

package/lib/index.browser.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-prototype-builtins */
 // Copyright (c) 2022 Marten Richter or other contributers (see commit). All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

package/lib/index.node.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-prototype-builtins */
 // Copyright (c) 2022 Marten Richter or other contributers (see commit). All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

package/lib/index.types.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-prototype-builtins */
 // Copyright (c) 2022 Marten Richter or other contributers (see commit). All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

package/lib/server.js

@@ -82,8 +82,8 @@ class TransportIntServerProxy {
   }
 
   /**
-   * @param {string} cert
-   * @param {string} privKey
+   * @param {string|string[]} cert
+   * @param {string|string[]} privKey
    * @param {boolean} http2only
    * */
   updateCert(cert, privKey, http2only) {
@@ -245,16 +245,18 @@ export class HttpServer {
    * @param {ServerSessionRequestEvent} args
    */
   onSessionRequest(args) {
-    if (args.promise && args.header) {
+    if ((args.promise || args.protocol !== 'http3:libquiche') && args.header) {
       if (!this.requestHandler) throw new Error('Request handler not set')
       this.requestHandler({ header: args.header })
         .then((/** @type {any} */ result) => {
           log('oSR', result)
-          args.object.finishSessionRequest({
+          this.transportInt.finishSessionRequest({
             promise: args.promise,
             header: args.header,
             session: args.session,
             head: args.head,
+            transportPrivate: args.transportPrivate,
+            protocol: args.protocol,
             ...result
           })
         })
@@ -272,6 +274,7 @@ export class HttpServer {
     const sesobj = new HttpWTSession({
       object: args.session,
       header: args.header,
+      userData: args.userData ?? {},
       parentobj: this
     })
     args.session.jsobj = sesobj

package/lib/session.js

@@ -50,7 +50,8 @@ export class HttpWTSession {
    * @param {object} args
    * @param {import('./types').NativeHttpWTSession} [args.object]
    * @param {HttpServer | HttpClient} args.parentobj
-   * @param {any | undefined} [args.header= undefined]
+   * @param {Object | undefined} [args.header= undefined]
+   * @param {Object | undefined} [args.userData= undefined]
    */
   constructor(args) {
     if (args.object) {
@@ -68,8 +69,10 @@ export class HttpWTSession {
     this.readyResolve = null
     /** @type {(() => void) | null | undefined} */
     this.closeHook = null
-    /** @type {(any | null | undefined)} */
+    /** @type {(Object | null | undefined)} */
     this.header = args.header
+    /** @type {(Object | null | undefined)} */
+    this.userData = args.userData
 
     /** @type {Promise<void>} */
     this.ready = new Promise((resolve, reject) => {
@@ -136,6 +139,7 @@ export class HttpWTSession {
         start: (controller) => {
           this.outgoDatagramController = controller
         },
+        // eslint-disable-next-line no-unused-vars
         write: (chunk, controller) => {
           if (this.state === 'closed') throw new Error('Session is closed')
           if (chunk instanceof Uint8Array) {
@@ -587,6 +591,7 @@ export class HttpWTSession {
   /**
    * @param {DatagramSendEvent} args
    */
+  // eslint-disable-next-line no-unused-vars
   onDatagramSend(args) {
     if (this.state === 'closed') return
     this.writeDatagramRej.shift()
@@ -601,6 +606,7 @@ export class HttpWTSession {
   /**
    * @param {GoawayReceivedEvent} args
    */
+  // eslint-disable-next-line no-unused-vars
   onGoAwayReceived(args) {
     if (this.drainingResolve) this.drainingResolve(undefined)
     this.state = 'draining'

package/lib/stream.js

@@ -52,9 +52,9 @@ export class HttpWTStream {
     this.pendingres = null
 
     /** @type {WebTransportReceiveStream} */
-    this.readable // eslint-disable-line no-unused-expressions
+    this.readable
     /** @type {WebTransportSendStream} */
-    this.writable // eslint-disable-line no-unused-expressions
+    this.writable
 
     /** @type {Promise<void> | null} */
     this.pendingoperationRead = null
@@ -69,12 +69,13 @@ export class HttpWTStream {
           this.objint.startReading()
         },
         pull: async (
+          // eslint-disable-next-line no-unused-vars
           /** @type {import("stream/web").ReadableByteStreamController} */ controller
         ) => {
           if (this.readableclosed) {
             return Promise.resolve()
           }
-
+          // eslint-disable-next-line no-unused-vars
           this.pendingoperationRead = new Promise((resolve, reject) => {
             this.pendingresRead = resolve
           })
@@ -83,6 +84,7 @@ export class HttpWTStream {
         },
         cancel: (/** @type {{ code: number; }} */ reason) => {
           /** @type {Promise<void>} */
+          // eslint-disable-next-line no-unused-vars
           const promise = new Promise((resolve, reject) => {
             this.cancelres = resolve
           })
@@ -128,6 +130,7 @@ export class HttpWTStream {
           start: (controller) => {
             this.writableController = controller
           },
+          // eslint-disable-next-line no-unused-vars
           write: (chunk, controller) => {
             if (this.writableclosed) {
               return Promise.resolve()
@@ -137,6 +140,7 @@ export class HttpWTStream {
               wchunk = new Uint8Array(wchunk)
             }
             if (wchunk instanceof Uint8Array) {
+              // eslint-disable-next-line no-unused-vars
               this.pendingoperation = new Promise((resolve, reject) => {
                 this.pendingres = resolve
               })
@@ -161,6 +165,7 @@ export class HttpWTStream {
               return Promise.resolve()
             }
             this.objint.streamFinal()
+            // eslint-disable-next-line no-unused-vars
             this.pendingoperation = new Promise((resolve, reject) => {
               this.pendingres = resolve
             })
@@ -168,6 +173,7 @@ export class HttpWTStream {
           },
           abort: (reason) => {
             if (this.writableclosed) {
+              // eslint-disable-next-line no-unused-vars
               return new Promise((resolve, reject) => {
                 resolve()
               })
@@ -179,6 +185,7 @@ export class HttpWTStream {
               else code = reason.code
             }
             /** @type {Promise<void>} */
+            // eslint-disable-next-line no-unused-vars
             const promise = new Promise((resolve, reject) => {
               this.abortres = resolve
             })
@@ -349,6 +356,7 @@ export class HttpWTStream {
   /**
    * @param {StreamWriteEvent} args
    */
+  // eslint-disable-next-line no-unused-vars
   onStreamWrite(args) {
     // we ignore success
     if (this.pendingoperation) {

package/lib/types.ts

@@ -4,7 +4,7 @@ import type {
   WebTransportOptions,
   WebTransportSendGroup
 } from './dom'
-import type { IncomingHttpHeaders, Http2Stream } from 'http2'
+import type { IncomingHttpHeaders, Http2Stream, ServerHttp2Stream } from 'http2'
 import { ParserBase } from './http2/parserbase'
 import { Http2WebTransportSession } from './http2/session'
 import { HttpClient } from './client'
@@ -103,11 +103,14 @@ export interface ReadBuffer {
 }
 
 export interface NativeFinishSessionRequest {
+  path: string
   header: IncomingHttpHeaders
-  session: Http2Stream
+  session: ServerHttp2Stream
   status: number
   protocol: 'capsule' | 'websocket' | 'websocketoverhttp1' | 'http3'
   head?: Buffer
+  userData?: object
+  transportPrivate?: object
 }
 
 export type Purpose =
@@ -219,6 +222,7 @@ export interface HttpWTServerSessionVisitorEvent
   extends HttpWTSessionVisitorEvent {
   path: string
   header: Object
+  userData?: Object | undefined
 }
 
 export interface ServerSessionRequestEvent {
@@ -228,6 +232,7 @@ export interface ServerSessionRequestEvent {
   session: any
   object?: any // the actual transport object itself, actually present on all messages, but required here
   protocol: string //'capsule' | 'websocket' | 'http3'
+  transportPrivate?: Object //private information from the transport object
 }
 
 /**
@@ -291,8 +296,8 @@ export interface HttpServerInit extends HttpWebTransportInit {
   port: string | number
   host: string
   secret: string
-  cert: string
-  privKey: string
+  cert: string | string[]
+  privKey: string | string[]
   maxConnections?: number
   initialStreamFlowControlWindow?: number
   streamShouldAutoTuneReceiveWindow?: boolean

package/lib/webtransport.browser.js

@@ -37,6 +37,7 @@ if (globalThis.WebTransport) {
       .then(() => {
         try {
           transport.close()
+          // eslint-disable-next-line no-empty, no-unused-vars
         } catch (error) {}
       })
       .catch(() => {})
@@ -61,9 +62,10 @@ export class WebTransportPonyfill extends WebTransportBase {
    * @param{{client: HttpClient, sessionint: HttpWTSession, ourl: URL}} args
    */
   startUpConnection({ client, sessionint, ourl }) {
+    const path = ourl.pathname + (ourl.search ?? '')
     client
-      .handleConnection({ createTransport: true, path: ourl.pathname })
-      .then(() => client.createWTSession(sessionint, ourl.pathname))
+      .handleConnection({ createTransport: true, path })
+      .then(() => client.createWTSession(sessionint, path))
       .catch((error) => {
         client.closeHookSession()
         sessionint.readyReject(error)
@@ -82,6 +84,7 @@ export class WebTransportPonyfill extends WebTransportBase {
   createClient(args) {
     this.curtype = 'websocket'
     const client = new HttpClient({
+      // eslint-disable-next-line no-unused-vars
       createReliableClient: (client) => {
         // @ts-ignore
         return new Http2WebTransportBrowser({ ...args })
@@ -214,8 +217,8 @@ export class WebTransportPolyfill {
     // @ts-ignore
     this.datagrams = {}
     // @ts-ignore
-    // eslint-disable-next-line no-undef
     this.datagrams.readable = new ReadableStream({
+      // eslint-disable-next-line no-unused-vars
       start: async (controller) => {
         await this.ready
         this.datagramsReader = this.curtransport.datagrams.readable.getReader()
@@ -230,12 +233,13 @@ export class WebTransportPolyfill {
       }
     })
     // @ts-ignore
-    // eslint-disable-next-line no-undef
     this.datagrams.writable = new WritableStream({
+      // eslint-disable-next-line no-unused-vars
       start: async (controller) => {
         await this.ready
         this.datagramsWriter = this.curtransport.datagrams.writable.getWriter()
       },
+      // eslint-disable-next-line no-unused-vars
       write: async (chunk, controller) => {
         await this.datagramsWriter.write(chunk)
       },
@@ -246,8 +250,8 @@ export class WebTransportPolyfill {
         await this.datagramsWriter.close()
       }
     })
-    // eslint-disable-next-line no-undef
     this.incomingBidirectionalStreams = new ReadableStream({
+      // eslint-disable-next-line no-unused-vars
       start: async (controller) => {
         await this.ready
         this.incomingBidirectionalStreamsReader =
@@ -263,8 +267,8 @@ export class WebTransportPolyfill {
         await this.incomingBidirectionalStreamsReader.cancel(reason)
       }
     })
-    // eslint-disable-next-line no-undef
     this.incomingUnidirectionalStreams = new ReadableStream({
+      // eslint-disable-next-line no-unused-vars
       start: async (controller) => {
         await this.ready
         this.incomingUnidirectionalStreamsReader =

package/lib/webtransport.node.js

@@ -23,7 +23,6 @@ let Http3WebTransportClient
  */
 let quicheLoadedReady
 // @ts-ignore
-// eslint-disable-next-line no-unused-vars
 export const quicheLoaded = new Promise((resolve, reject) => {
   // @ts-ignore
   import('@fails-components/webtransport-transport-http3-quiche')
@@ -75,17 +74,18 @@ export class WebTransport extends WebTransportBase {
    * @param{{client: HttpClient, sessionint: HttpWTSession, ourl: URL}} args
    */
   startUpConnection({ client, sessionint, ourl }) {
+    const path = ourl.pathname + (ourl.search ?? '')
     client
-      .handleConnection({ createTransport: true, path: ourl.pathname })
-      .then(() => client.createWTSession(sessionint, ourl.pathname))
+      .handleConnection({ createTransport: true, path })
+      .then(() => client.createWTSession(sessionint, path))
       .catch((error) => {
         if (client.transportIntSwitchToReliable) {
           log('Connecting to unreliable failed:', error)
           log('Now switching to reliable')
           client.transportIntSwitchToReliable()
           client
-            .handleConnection({ createTransport: false, path: ourl.pathname })
-            .then(() => client.createWTSession(sessionint, ourl.pathname))
+            .handleConnection({ createTransport: false, path })
+            .then(() => client.createWTSession(sessionint, path))
             .catch((error) => {
               client.closeHookSession()
               sessionint.readyReject(error)
@@ -109,9 +109,11 @@ export class WebTransport extends WebTransportBase {
    */
   createClient(args) {
     let createUnreliableClient
-    if (!quicheLoadedReady)
+    // @ts-ignore
+    if (!quicheLoadedReady && !args?.forceReliable)
       throw new Error('Lib quiche loading attempt did not end')
     if (checkQuicheInit) {
+      // eslint-disable-next-line no-unused-vars
       createUnreliableClient = function (/** @type {any} */ _client) {
         if (
           // @ts-ignore
@@ -130,6 +132,7 @@ export class WebTransport extends WebTransportBase {
       }
     }
     const client = new HttpClient({
+      // eslint-disable-next-line no-unused-vars
       createReliableClient: function (client) {
         // @ts-ignore
         return new Http2WebTransportClient({ ...args })

package/lib/webtransportbase.js

@@ -59,6 +59,7 @@ export class WebTransportBase {
    * @return {{sessionint: HttpWTSession, client: HttpClient}}
    * @abstract
    */
+  // eslint-disable-next-line no-unused-vars
   createClient(args) {
     throw new Error('Implement createClient')
   }
@@ -67,6 +68,7 @@ export class WebTransportBase {
    * @param{{client: HttpClient, sessionint: HttpWTSession, ourl: URL}} args
    * @abstract
    */
+  // eslint-disable-next-line no-unused-vars
   startUpConnection({ client, sessionint, ourl }) {
     throw new Error('Implement startUpConnection')
   }

package/old_test/echoclient.js

@@ -19,7 +19,6 @@ async function startClientTests(args, hashes) {
       value: Buffer.from(el.value.split(':').map((el) => parseInt(el, 16)))
     }))
   }
-  // eslint-disable-next-line no-undef
   console.log('hashagrs', hashargs)
   const transport = new WebTransport(url, hashargs)
   transport.closed

package/old_test/testsuite.js

@@ -147,7 +147,7 @@ export async function echoTestsConnection(transport) {
   const resultArray1 = new Uint8Array(i)
   console.log('TEST 1: start')
 
-  while (true && i > 0) {
+  while (i > 0) {
     const { done, value } = await reader.read()
     if (done) {
       break
@@ -199,7 +199,7 @@ export async function echoTestsConnection(transport) {
     refArray2.set(data4, data3.length)
 
     const resultArray2 = new Uint8Array(i)
-    while (true && i > 0) {
+    while (i > 0) {
       const { done, value } = await readbd.read()
       if (done) {
         break
@@ -255,7 +255,7 @@ export async function echoTestsConnection(transport) {
 
     const resultArray3 = new Uint8Array(i)
 
-    while (true && i > 0) {
+    while (i > 0) {
       const { done, value } = await readud.read()
       if (done) {
         break
@@ -302,7 +302,7 @@ export async function echoTestsConnection(transport) {
   pos = 0
   const resultArray4 = new Uint8Array(i)
 
-  while (true && i > 0) {
+  while (i > 0) {
     const { done, value } = await readdg.read()
     if (done) {
       break