@stream-io/video-client 1.0.3 → 1.0.4

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 This file was generated using [@jscutlery/semver](https://github.com/jscutlery/semver).
 
+### [1.0.4](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.0.3...@stream-io/video-client-1.0.4) (2024-05-14)
+
+
+### Bug Fixes
+
+* don't create publisher PC for anonymous users ([#1353](https://github.com/GetStream/stream-video-js/issues/1353)) ([7331767](https://github.com/GetStream/stream-video-js/commit/7331767bd9254082517b1f36895796032b7af149))
+
 ### [1.0.3](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.0.2...@stream-io/video-client-1.0.3) (2024-05-13)
 
 

package/dist/index.browser.es.js

@@ -9788,25 +9788,29 @@ const flatten = (report) => {
 };
 const getSdkSignature = (clientDetails) => {
     const { sdk, ...platform } = clientDetails;
-    const sdkName = sdk && sdk.type === SdkType.REACT
-        ? 'stream-react'
-        : sdk && sdk.type === SdkType.REACT_NATIVE
-            ? 'stream-react-native'
-            : 'stream-js';
-    const sdkVersion = sdk
-        ? `${sdk.major}.${sdk.minor}.${sdk.patch}`
-        : '0.0.0-development';
+    const sdkName = getSdkName(sdk);
+    const sdkVersion = getSdkVersion(sdk);
     return {
         sdkName,
         sdkVersion,
         ...platform,
     };
 };
+const getSdkName = (sdk) => {
+    return sdk && sdk.type === SdkType.REACT
+        ? 'stream-react'
+        : sdk && sdk.type === SdkType.REACT_NATIVE
+            ? 'stream-react-native'
+            : 'stream-js';
+};
+const getSdkVersion = (sdk) => {
+    return sdk ? `${sdk.major}.${sdk.minor}.${sdk.patch}` : '0.0.0-development';
+};
 
 /**
  * Creates a new StatsReporter instance that collects metrics about the ongoing call and reports them to the state store
  */
-const createStatsReporter = ({ subscriber, publisher, state, pollingIntervalInMs = 2000, }) => {
+const createStatsReporter = ({ subscriber, publisher, state, datacenter, pollingIntervalInMs = 2000, }) => {
     const logger = getLogger(['stats']);
     const getRawStatsForTrack = async (kind, selector) => {
         if (kind === 'subscriber' && subscriber) {
@@ -9816,12 +9820,13 @@ const createStatsReporter = ({ subscriber, publisher, state, pollingIntervalInMs
             return publisher.getStats(selector);
         }
         else {
-            logger('warn', `Can't retrieve RTC stats for ${kind}`);
             return undefined;
         }
     };
     const getStatsForStream = async (kind, mediaStream) => {
         const pc = kind === 'subscriber' ? subscriber : publisher;
+        if (!pc)
+            return [];
         const statsForStream = [];
         for (let track of mediaStream.getTracks()) {
             const report = await pc.getStats(track);
@@ -9867,7 +9872,7 @@ const createStatsReporter = ({ subscriber, publisher, state, pollingIntervalInMs
                     });
                 }
                 catch (e) {
-                    logger('error', `Failed to collect stats for ${kind} if ${participant.userId}`, e);
+                    logger('error', `Failed to collect stats for ${kind} of ${participant.userId}`, e);
                 }
             }
         }
@@ -9880,19 +9885,21 @@ const createStatsReporter = ({ subscriber, publisher, state, pollingIntervalInMs
             }))
                 .then(aggregate),
             publisher
-                .getStats()
-                .then((report) => transform(report, {
-                kind: 'publisher',
-                trackKind: 'video',
-            }))
-                .then(aggregate),
+                ? publisher
+                    .getStats()
+                    .then((report) => transform(report, {
+                    kind: 'publisher',
+                    trackKind: 'video',
+                }))
+                    .then(aggregate)
+                : getEmptyStats(),
         ]);
         const [subscriberRawStats, publisherRawStats] = await Promise.all([
             getRawStatsForTrack('subscriber'),
-            getRawStatsForTrack('publisher'),
+            publisher ? getRawStatsForTrack('publisher') : undefined,
         ]);
         state.setCallStatsReport({
-            datacenter: publisher.sfuClient.edgeName,
+            datacenter,
             publisherStats,
             subscriberStats,
             subscriberRawStats,
@@ -9969,14 +9976,9 @@ const transform = (report, opts) => {
         timestamp: Date.now(),
     };
 };
-/**
- * Aggregates generic stats.
- *
- * @param stats the stats to aggregate.
- */
-const aggregate = (stats) => {
-    const aggregatedStats = {
-        rawReport: stats,
+const getEmptyStats = (stats) => {
+    return {
+        rawReport: stats ?? { streams: [], timestamp: Date.now() },
         totalBytesSent: 0,
         totalBytesReceived: 0,
         averageJitterInMs: 0,
@@ -9987,6 +9989,14 @@ const aggregate = (stats) => {
         highestFramesPerSecond: 0,
         timestamp: Date.now(),
     };
+};
+/**
+ * Aggregates generic stats.
+ *
+ * @param stats the stats to aggregate.
+ */
+const aggregate = (stats) => {
+    const aggregatedStats = getEmptyStats(stats);
     let maxArea = -1;
     const area = (w, h) => w * h;
     const qualityLimitationReasons = new Set();
@@ -10030,7 +10040,7 @@ class SfuStatsReporter {
         this.run = async () => {
             const [subscriberStats, publisherStats] = await Promise.all([
                 this.subscriber.getStats().then(flatten).then(JSON.stringify),
-                this.publisher.getStats().then(flatten).then(JSON.stringify),
+                this.publisher?.getStats().then(flatten).then(JSON.stringify) ?? '[]',
             ]);
             await this.sfuClient.sendStats({
                 sdk: this.sdkName,
@@ -10059,15 +10069,8 @@ class SfuStatsReporter {
         this.publisher = publisher;
         const webRTCInfo = getWebRTCInfo();
         const { sdk, browser } = clientDetails;
-        this.sdkName =
-            sdk && sdk.type === SdkType.REACT
-                ? 'stream-react'
-                : sdk && sdk.type === SdkType.REACT_NATIVE
-                    ? 'stream-react-native'
-                    : 'stream-js';
-        this.sdkVersion = sdk
-            ? `${sdk.major}.${sdk.minor}.${sdk.patch}`
-            : '0.0.0-development';
+        this.sdkName = getSdkName(sdk);
+        this.sdkVersion = getSdkVersion(sdk);
         // The WebRTC version if passed from the SDK, it is taken else the browser info is sent.
         this.webRTCVersion =
             webRTCInfo?.version ||
@@ -12623,7 +12626,10 @@ class Call {
                     connectionConfig,
                 });
             }
-            if (!this.publisher) {
+            // anonymous users can't publish anything hence, there is no need
+            // to create Publisher Peer Connection for them
+            const isAnonymous = this.streamClient.user?.type === 'anonymous';
+            if (!this.publisher && !isAnonymous) {
                 const audioSettings = this.state.settings?.audio;
                 const isDtxEnabled = !!audioSettings?.opus_dtx_enabled;
                 const isRedEnabled = !!audioSettings?.redundant_coding_enabled;
@@ -12641,6 +12647,7 @@ class Call {
                     subscriber: this.subscriber,
                     publisher: this.publisher,
                     state: this.state,
+                    datacenter: this.sfuClient.edgeName,
                 });
             }
             const clientDetails = getClientDetails();
@@ -12686,15 +12693,18 @@ class Call {
                 }
                 if (isMigrating) {
                     await this.subscriber.migrateTo(sfuClient, connectionConfig);
-                    await this.publisher.migrateTo(sfuClient, connectionConfig);
+                    await this.publisher?.migrateTo(sfuClient, connectionConfig);
                 }
                 else if (isReconnecting) {
                     if (reconnected) {
                         // update the SFU client instance on the subscriber and publisher
                         this.subscriber.setSfuClient(sfuClient);
-                        this.publisher.setSfuClient(sfuClient);
-                        // and perform a full ICE restart on the publisher
-                        await this.publisher.restartIce();
+                        // publisher might not be there (anonymous users)
+                        if (this.publisher) {
+                            this.publisher.setSfuClient(sfuClient);
+                            // and perform a full ICE restart on the publisher
+                            await this.publisher.restartIce();
+                        }
                     }
                     else if (previousSfuClient?.isFastReconnecting) {
                         // reconnection wasn't possible, so we need to do a full rejoin
@@ -15193,7 +15203,7 @@ class StreamClient {
             });
         };
         this.getUserAgent = () => {
-            const version = "1.0.3" ;
+            const version = "1.0.4" ;
             return (this.userAgent ||
                 `stream-video-javascript-client-${this.node ? 'node' : 'browser'}-${version}`);
         };