@reactoo/watchtogether-sdk-js 2.8.40 → 2.8.41

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reactoo/watchtogether-sdk-js",
-  "version": "2.8.40",
+  "version": "2.8.41",
   "description": "Javascript SDK for Reactoo",
   "main": "dist/watchtogether-sdk.min.js",
   "module": "dist/watchtogether-sdk.min.js",

package/src/modules/wt-room-sfu.js

@@ -253,7 +253,6 @@ class RoomSession {
 		this.simulcastSettings = null;
 		this.useWebsockets = null;
 
-
 		this.defaultSimulcastSettings = {
 			"default" : {
 				mode: "controlled", // controlled, manual, browserControlled
@@ -662,7 +661,11 @@ class RoomSession {
 
 	async #updateSubscriptions() {
 
-		// no subscriber handle, we create one and subscribe to relevant participants
+
+		// added sanity check
+		await this.#availablePublishersSanityCheck();
+
+		// no subscription yet, we create one and subscribe to relevant participants
 
 		if(!this.#subscriberJoinPromise) {
 			await this.#joinAsSubscriber(this.roomId,  this.pin, this.userId);
@@ -818,6 +821,43 @@ class RoomSession {
 		}
 	}
 
+	async #availablePublishersSanityCheck() {
+		const list = await this.sendMessage(this.handleId, {body: {request: 'listparticipants', room: this.roomId}})
+		const participants = list.participants || [];
+		const availablePublishers = [...(this.#subscriberHandle?.webrtcStuff?.availablePublishers ?? [])];
+
+		availablePublishers.forEach(publisher => {
+
+			const checkedParticipant = participants.find(p => p.id === publisher.id);
+			if(!checkedParticipant) {
+				// this publisher is not in the list, we need to remove it
+				this._log('Sanity check - Removing publisher', publisher.id, 'from availablePublishers');
+				this.#updateAvailablePublishersTrackData([], publisher.id);
+				this.#emitRemoteFeedUpdate(null, {feedRemoval: true, attendee: {id: publisher.id}});
+				this.#removeAttendeeFromCache(publisher.id);
+				this.emit('error', {
+					type: 'warning',
+					id: 50,
+					message: 'sanity check - removing publisher from availablePublishers',
+					data: publisher.id
+				});
+			}
+			else if(checkedParticipant.publisher === false) {
+				// this publisher is not in the list, we need to remove it
+				this._log('Sanity check - Removing publisher', publisher.id, 'from availablePublishers');
+				this.#updateAvailablePublishersTrackData([], publisher.id);
+				this.#emitRemoteFeedUpdate(null, {feedRemoval: true, attendee: {id: publisher.id}});
+				this.emit('error', {
+					type: 'warning',
+					id: 51,
+					message: 'sanity check - removing publisher from availablePublishers',
+					data: publisher.id
+				});
+			}
+
+		});
+	}
+
 	#getTransceiverDataByMid(mid) {
 		return this.#subscriberHandle.webrtcStuff.transceiverMap.find(t => t.mid === mid);
 	}