@stream-io/video-client 1.0.2 → 1.0.4

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 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)
+
+
+### Bug Fixes
+
+* improve error handling across the SDK ([#1350](https://github.com/GetStream/stream-video-js/issues/1350)) ([ac0ae3b](https://github.com/GetStream/stream-video-js/commit/ac0ae3b7d5da91152d0f41a203b73e6c99c42ff9))
+
 ### [1.0.2](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.0.1...@stream-io/video-client-1.0.2) (2024-05-13)
 
 

package/dist/index.browser.es.js

@@ -8399,15 +8399,19 @@ class Publisher {
                 return getPreferredCodecs('audio', preferredCodec ?? defaultAudioCodec, codecToRemove);
             }
         };
-        this.onIceCandidate = async (e) => {
+        this.onIceCandidate = (e) => {
             const { candidate } = e;
             if (!candidate) {
                 logger$3('debug', 'null ice candidate');
                 return;
             }
-            await this.sfuClient.iceTrickle({
+            this.sfuClient
+                .iceTrickle({
                 iceCandidate: getIceCandidate(candidate),
                 peerType: PeerType.PUBLISHER_UNSPECIFIED,
+            })
+                .catch((err) => {
+                logger$3('warn', `ICETrickle failed`, err);
             });
         };
         /**
@@ -8448,8 +8452,8 @@ class Publisher {
             }
             await this.negotiate({ iceRestart: true });
         };
-        this.onNegotiationNeeded = async () => {
-            await this.negotiate();
+        this.onNegotiationNeeded = () => {
+            this.negotiate().catch((err) => logger$3('warn', `Negotiation failed.`, err));
         };
         /**
          * Initiates a new offer/answer exchange with the currently connected SFU.
@@ -8639,10 +8643,12 @@ class Publisher {
         this.isDtxEnabled = isDtxEnabled;
         this.isRedEnabled = isRedEnabled;
         this.iceRestartDelay = iceRestartDelay;
-        this.unsubscribeOnIceRestart = dispatcher.on('iceRestart', async (iceRestart) => {
+        this.unsubscribeOnIceRestart = dispatcher.on('iceRestart', (iceRestart) => {
             if (iceRestart.peerType !== PeerType.PUBLISHER_UNSPECIFIED)
                 return;
-            await this.restartIce();
+            this.restartIce().catch((err) => {
+                logger$3('warn', `ICERestart failed`, err);
+            });
         });
     }
 }
@@ -8836,15 +8842,19 @@ class Subscriber {
                 [streamKindProp]: primaryStream,
             });
         };
-        this.onIceCandidate = async (e) => {
+        this.onIceCandidate = (e) => {
             const { candidate } = e;
             if (!candidate) {
                 logger$2('debug', 'null ice candidate');
                 return;
             }
-            await this.sfuClient.iceTrickle({
+            this.sfuClient
+                .iceTrickle({
                 iceCandidate: getIceCandidate(candidate),
                 peerType: PeerType.SUBSCRIBER,
+            })
+                .catch((err) => {
+                logger$2('warn', `ICETrickle failed`, err);
             });
         };
         this.negotiate = async (subscriberOffer) => {
@@ -8916,13 +8926,17 @@ class Subscriber {
         this.state = state;
         this.iceRestartDelay = iceRestartDelay;
         this.pc = this.createPeerConnection(connectionConfig);
-        this.unregisterOnSubscriberOffer = dispatcher.on('subscriberOffer', async (subscriberOffer) => {
-            await this.negotiate(subscriberOffer);
+        this.unregisterOnSubscriberOffer = dispatcher.on('subscriberOffer', (subscriberOffer) => {
+            this.negotiate(subscriberOffer).catch((err) => {
+                logger$2('warn', `Negotiation failed.`, err);
+            });
         });
-        this.unregisterOnIceRestart = dispatcher.on('iceRestart', async (iceRestart) => {
+        this.unregisterOnIceRestart = dispatcher.on('iceRestart', (iceRestart) => {
             if (iceRestart.peerType !== PeerType.SUBSCRIBER)
                 return;
-            await this.restartIce();
+            this.restartIce().catch((err) => {
+                logger$2('warn', `ICERestart failed`, err);
+            });
         });
     }
 }
@@ -9774,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) {
@@ -9802,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);
@@ -9853,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);
                 }
             }
         }
@@ -9866,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,
@@ -9955,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,
@@ -9973,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();
@@ -10016,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,
@@ -10045,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 ||
@@ -12609,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;
@@ -12627,6 +12647,7 @@ class Call {
                     subscriber: this.subscriber,
                     publisher: this.publisher,
                     state: this.state,
+                    datacenter: this.sfuClient.edgeName,
                 });
             }
             const clientDetails = getClientDetails();
@@ -12672,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
@@ -13488,7 +13512,9 @@ class Call {
         });
         this.leaveCallHooks.add(registerEventHandlers(this, this.state, this.dispatcher));
         this.registerEffects();
-        this.leaveCallHooks.add(createSubscription(this.trackSubscriptionsSubject.pipe(debounce((v) => timer(v.type)), map$1((v) => v.data)), (subscriptions) => this.sfuClient?.updateSubscriptions(subscriptions)));
+        this.leaveCallHooks.add(createSubscription(this.trackSubscriptionsSubject.pipe(debounce((v) => timer(v.type)), map$1((v) => v.data)), (subscriptions) => this.sfuClient?.updateSubscriptions(subscriptions).catch((err) => {
+            this.logger('debug', `Failed to update track subscriptions`, err);
+        })));
         this.camera = new CameraManager(this);
         this.microphone = new MicrophoneManager(this);
         this.speaker = new SpeakerManager(this);
@@ -15177,7 +15203,7 @@ class StreamClient {
             });
         };
         this.getUserAgent = () => {
-            const version = "1.0.2" ;
+            const version = "1.0.4" ;
             return (this.userAgent ||
                 `stream-video-javascript-client-${this.node ? 'node' : 'browser'}-${version}`);
         };