@livedigital/client 2.7.1 → 2.7.2-improve-network-issue-detector.1

package/package.json

@@ -2,7 +2,7 @@
   "name": "@livedigital/client",
   "author": "vlprojects",
   "license": "MIT",
-  "version": "2.7.1",
+  "version": "2.7.2-improve-network-issue-detector.1",
   "private": false,
   "bugs": {
     "url": "https://github.com/vlprojects/livedigital-sdk/issues"

package/src/engine/wid/detectors/NetworkIssueDetector.ts

@@ -30,21 +30,30 @@ class NetworkIssueDetector implements IssueDetector {
 
     const rtpNetworkStats = inboundRTPStreamsStats.reduce((stats, currentStreamStats) => {
       const previousStreamStats = previousInboundStreamStats.find((stream) => stream.ssrc === currentStreamStats.ssrc);
+
       const lastJitterBufferDelay = previousStreamStats?.jitterBufferDelay || 0;
       const lastJitterBufferEmittedCount = previousStreamStats?.jitterBufferEmittedCount || 0;
       const delay = currentStreamStats.jitterBufferDelay - lastJitterBufferDelay;
       const emitted = currentStreamStats.jitterBufferEmittedCount - lastJitterBufferEmittedCount;
-      const latency = delay && emitted ? (1e3 * delay) / emitted : 0;
+      const jitterBufferDelayMs = delay && emitted ? (1e3 * delay) / emitted : 0;
 
       return {
-        maxLatency: latency > stats.maxLatency ? latency : stats.maxLatency,
+        sumJitter: stats.sumJitter + currentStreamStats.jitter,
+        sumJitterBufferDelayMs: stats.sumJitterBufferDelayMs + jitterBufferDelayMs,
         packetsLost: stats.packetsLost + currentStreamStats.packetsLost,
         lastPacketsLost: stats.lastPacketsLost + (previousStreamStats?.packetsLost || 0),
       };
-    }, { maxLatency: 0, packetsLost: 0, lastPacketsLost: 0 });
+    }, {
+      sumJitter: 0,
+      sumJitterBufferDelayMs: 0,
+      packetsLost: 0,
+      lastPacketsLost: 0,
+    });
 
     const rtt = (1e3 * data.connection.currentRoundTripTime) || 0;
-    const jitter = Math.round(rtt / 2 + rtpNetworkStats.maxLatency);
+    const { sumJitter, sumJitterBufferDelayMs } = rtpNetworkStats;
+    const avgJitter = sumJitter / inboundRTPStreamsStats.length;
+    const avgJitterBufferDelay = sumJitterBufferDelayMs / inboundRTPStreamsStats.length;
 
     const deltaPacketReceived = packetsReceived - lastPacketsReceived;
     const deltaPacketLost = rtpNetworkStats.packetsLost - rtpNetworkStats.lastPacketsLost;
@@ -54,19 +63,24 @@ class NetworkIssueDetector implements IssueDetector {
       : 0;
 
     const isHighPacketsLoss = packetsLoss > 5;
-    const isHighJitter = jitter >= 200;
-    const isHighRTT = rtt >= 200;
+    const isHighJitter = avgJitter >= 200;
+    const isHighRTT = rtt >= 250;
+    const isHighJitterBufferDelay = avgJitterBufferDelay > 500;
 
     const isNetworkIssue = (!isHighPacketsLoss && isHighJitter) || isHighJitter || isHighPacketsLoss;
     const isServerIssue = isHighRTT && !isHighJitter && !isHighPacketsLoss;
     const isNetworkMediaLatencyIssue = isHighPacketsLoss && isHighJitter;
+    const isNetworkMediaSyncIssue = isHighJitter && isHighJitterBufferDelay;
+
+    const debug = `packetLoss: ${packetsLoss}%, jitter: ${avgJitter}, rtt: ${rtt},`
+      + ` jitterBuffer: ${avgJitterBufferDelay}ms`;
 
     if (isNetworkIssue) {
       issues.push({
         type: IssueType.Network,
         reason: IssueReason.NetworkQuality,
         iceCandidate: data.connection.local.id,
-        debug: `packetLoss: ${packetsLoss}%, jitter: ${jitter}`,
+        debug,
       });
     }
 
@@ -75,7 +89,7 @@ class NetworkIssueDetector implements IssueDetector {
         type: IssueType.Server,
         reason: IssueReason.ServerIssue,
         iceCandidate: data.connection.remote.id,
-        debug: `packetLoss: ${packetsLoss}%,jitter: ${jitter}, rtt: ${rtt},`,
+        debug,
       });
     }
 
@@ -84,7 +98,16 @@ class NetworkIssueDetector implements IssueDetector {
         type: IssueType.Network,
         reason: IssueReason.NetworkMediaLatency,
         iceCandidate: data.connection.local.id,
-        debug: `packetLoss: ${packetsLoss}%, jitter: ${jitter}`,
+        debug,
+      });
+    }
+
+    if (isNetworkMediaSyncIssue) {
+      issues.push({
+        type: IssueType.Network,
+        reason: IssueReason.NetworkMediaSyncFailure,
+        iceCandidate: data.connection.local.id,
+        debug,
       });
     }