@livedigital/client 3.44.2 → 3.44.4

package/package.json

@@ -2,7 +2,7 @@
   "name": "@livedigital/client",
   "author": "vlprojects",
   "license": "MIT",
-  "version": "3.44.2",
+  "version": "3.44.4",
   "private": false,
   "bugs": {
     "url": "https://github.com/vlprojects/livedigital-sdk/issues"

package/src/engine/index.ts

@@ -76,7 +76,7 @@ class Engine {
   private readonly mark = randomString();
 
   private readonly handleSignalingError = (error: Error): void => {
-    if (this.isReleasing) {
+    if (this.isReleasing || !this.isChannelJoined || this.state.isChannelJoining) {
       return;
     }
 
@@ -135,7 +135,6 @@ class Engine {
       const [sendTransport, recvTransport] = await Promise.all([
         this.network.createSendTransport(this.media.mediasoupDevice),
         this.network.createRecvTransport(this.media.mediasoupDevice),
-        // @INFO: un-comment to enable signaling tcp transport connection
         this.network.createSignalingTransport(this.media.mediasoupDevice),
       ]);
 
@@ -160,9 +159,8 @@ class Engine {
         recvTransportId: this.network.receiveTransport?.id,
       });
 
-      // @INFO: un-comment to enable signaling tcp transport connection
-      // @WARN: must be non-blocking, no await
-      await this.network.createSignalingConsumer();
+      await this.network.createHeartbeatProducer();
+      await this.network.createKeepAliveConsumer();
     } catch (error: unknown) {
       this.#logger.error('initialize()', { error });
 
@@ -194,6 +192,7 @@ class Engine {
       this.state.reset();
       this.myPeer.delete();
       this.webRtcIssueDetector?.stopWatchingNewPeerConnections();
+
       await this.peers.reset();
 
       this.network.closeSendTransport();
@@ -212,18 +211,8 @@ class Engine {
 
       this.removeClientEventEmitterListeners();
 
-      if (this.isChannelJoined) {
-        try {
-          await this.signalingApiClient.peers.disconnectPeer();
-        } catch (error: unknown) {
-          this.#logger.debug('failed to disconnect peer', {
-            peerId: this.myPeerId,
-            error: serializeError(error),
-          });
-        }
-      }
-
       this.network.reset();
+      await this.signalingApiClient.peers.disconnectPeer();
       clientMetaProvider.clear();
       this.transportsStatsProvider.clear();
       this.statsHandler.clear();
@@ -232,6 +221,7 @@ class Engine {
       });
     } catch (error: unknown) {
       this.#logger.error('release()', { error });
+      throw new Error('Error release engine');
     } finally {
       this.isReleasing = false;
     }
@@ -675,7 +665,7 @@ class Engine {
   }
 
   private triggerRejoinEventIfAppropriate(reason?: string): void {
-    if (this.rejoinRequested && !this.state.isChannelJoining) {
+    if (!this.isChannelJoined || this.isReleasing || this.state.isChannelJoining || this.rejoinRequested) {
       return;
     }
 

package/src/engine/network/index.ts

@@ -62,7 +62,7 @@ class Network {
 
   public receiveTransport?: Transport;
 
-  public signalingDataProducer?: types.DataProducer;
+  public heartbeatDataProducer?: types.DataProducer;
 
   public signalingTransport?: Transport;
 
@@ -132,7 +132,7 @@ class Network {
 
   public reset(): void {
     this.signalingHeartbeater.dispose();
-    this.signalingDataProducer = undefined;
+    this.heartbeatDataProducer = undefined;
     this.stopRestartIceAttempts();
     this.removeEventListeners();
   }
@@ -333,12 +333,12 @@ class Network {
     };
   }
 
-  async createSignalingConsumer(): Promise<void> {
-    const transport = this.signalingTransport;
+  async createHeartbeatProducer(): Promise<void> {
+    const transport = this.sendTransport;
 
     if (!transport) {
-      this.#logger.error('createSignalingConsumer()', { reason: 'missing signaling transport' });
-      throw new Error('Failed to create data consumer');
+      this.#logger.error('createHeartbeatProducer()', { reason: 'missing send transport' });
+      throw new Error('Failed to create data producer');
     }
 
     const dataProducer = await transport.produceData({
@@ -346,17 +346,41 @@ class Network {
       protocol: 'heartbeat',
     });
 
-    this.signalingDataProducer = dataProducer;
+    this.heartbeatDataProducer = dataProducer;
 
     dataProducer.once('transportclose', () => {
-      this.signalingDataProducer?.close();
-      this.signalingDataProducer = undefined;
+      this.heartbeatDataProducer?.close();
+      this.heartbeatDataProducer = undefined;
       this.signalingHeartbeater.dispose();
     });
 
     this.signalingHeartbeater.init(transport, dataProducer);
   }
 
+  // only for tcp transport connection
+  async createKeepAliveConsumer(): Promise<void> {
+    const transport = this.signalingTransport;
+
+    if (!transport) {
+      this.#logger.error('createKeepAliveConsumer()', { reason: 'missing signaling transport' });
+
+      return;
+    }
+
+    const dataProducer = await transport.produceData({
+      label: 'consumers-scores',
+      protocol: 'keepalive',
+    });
+
+    dataProducer.once('open', () => {
+      dataProducer.send(new Uint8Array(0));
+    });
+
+    dataProducer.once('transportclose', () => {
+      dataProducer.close();
+    });
+  }
+
   get channelMessagesDataConsumer(): types.DataConsumer | undefined {
     return this.dataChannelsManager.channelMessagesDataConsumer;
   }