@webex/plugin-meetings 3.7.0-next.64 → 3.7.0-next.65

package/dist/webinar/index.js

@@ -458,7 +458,7 @@ var Webinar = _webexCore.WebexPlugin.extend({
       }, _callee7);
     }))();
   },
-  version: "3.7.0-next.64"
+  version: "3.7.0-next.65"
 });
 var _default = exports.default = Webinar;
 //# sourceMappingURL=index.js.map

package/package.json

@@ -43,7 +43,7 @@
     "@webex/eslint-config-legacy": "0.0.0",
     "@webex/jest-config-legacy": "0.0.0",
     "@webex/legacy-tools": "0.0.0",
-    "@webex/plugin-meetings": "3.7.0-next.64",
+    "@webex/plugin-meetings": "3.7.0-next.65",
     "@webex/plugin-rooms": "3.7.0-next.25",
     "@webex/test-helper-chai": "3.7.0-next.18",
     "@webex/test-helper-mocha": "3.7.0-next.18",
@@ -63,7 +63,7 @@
   "dependencies": {
     "@webex/common": "3.7.0-next.18",
     "@webex/event-dictionary-ts": "^1.0.1688",
-    "@webex/internal-media-core": "2.14.3",
+    "@webex/internal-media-core": "2.14.4",
     "@webex/internal-plugin-conversation": "3.7.0-next.25",
     "@webex/internal-plugin-device": "3.7.0-next.18",
     "@webex/internal-plugin-llm": "3.7.0-next.23",
@@ -71,8 +71,8 @@
     "@webex/internal-plugin-metrics": "3.7.0-next.18",
     "@webex/internal-plugin-support": "3.7.0-next.26",
     "@webex/internal-plugin-user": "3.7.0-next.18",
-    "@webex/internal-plugin-voicea": "3.7.0-next.64",
-    "@webex/media-helpers": "3.7.0-next.24",
+    "@webex/internal-plugin-voicea": "3.7.0-next.65",
+    "@webex/media-helpers": "3.7.0-next.25",
     "@webex/plugin-people": "3.7.0-next.23",
     "@webex/plugin-rooms": "3.7.0-next.25",
     "@webex/web-capabilities": "^1.4.0",
@@ -92,5 +92,5 @@
   "//": [
     "TODO: upgrade jwt-decode when moving to node 18"
   ],
-  "version": "3.7.0-next.64"
+  "version": "3.7.0-next.65"
 }

package/src/media/MediaConnectionAwaiter.ts

@@ -111,6 +111,8 @@ export default class MediaConnectionAwaiter {
 
     this.clearCallbacks();
 
+    LoggerProxy.logger.warn('Media:MediaConnectionAwaiter#connectionStateChange --> Resolving');
+
     this.defer.resolve();
   }
 

package/src/media/properties.ts

@@ -287,24 +287,45 @@ export default class MediaProperties {
     selectedCandidatePairChanges: number;
     numTransports: number;
   }> {
-    const allStatsReports = [];
-
     try {
-      const statsResult = await this.webrtcMediaConnection.getStats();
-      statsResult.forEach((report) => allStatsReports.push(report));
+      const allStatsReports = [];
+
+      await new Promise((resolve, reject) => {
+        const timeout = setTimeout(() => {
+          reject(new Error('timed out'));
+        }, 1000);
+
+        this.webrtcMediaConnection
+          .getStats()
+          .then((statsResult) => {
+            clearTimeout(timeout);
+            statsResult.forEach((report) => allStatsReports.push(report));
+            resolve(allStatsReports);
+          })
+          .catch((error) => {
+            clearTimeout(timeout);
+            reject(error);
+          });
+      });
+
+      const connectionType = this.getConnectionType(allStatsReports);
+      const {selectedCandidatePairChanges, numTransports} = this.getTransportInfo(allStatsReports);
+
+      return {
+        connectionType,
+        selectedCandidatePairChanges,
+        numTransports,
+      };
     } catch (error) {
       LoggerProxy.logger.warn(
         `Media:properties#getCurrentConnectionInfo --> getStats() failed: ${error}`
       );
-    }
-
-    const connectionType = this.getConnectionType(allStatsReports);
-    const {selectedCandidatePairChanges, numTransports} = this.getTransportInfo(allStatsReports);
 
-    return {
-      connectionType,
-      selectedCandidatePairChanges,
-      numTransports,
-    };
+      return {
+        connectionType: 'unknown',
+        selectedCandidatePairChanges: -1,
+        numTransports: 0,
+      };
+    }
   }
 }

package/src/meeting/index.ts

@@ -3431,6 +3431,10 @@ export default class Meeting extends StatelessWebexPlugin {
     // The second on is if the audio is muted, we need to tell the statsAnalyzer when
     // the audio is muted or the user is not willing to send media
     this.locusInfo.on(LOCUSINFO.EVENTS.MEDIA_STATUS_CHANGE, (status) => {
+      LoggerProxy.logger.info(
+        'Meeting:index#setUpLocusInfoSelfListener --> MEDIA_STATUS_CHANGE received, processing...'
+      );
+
       if (this.statsAnalyzer) {
         this.statsAnalyzer.updateMediaStatus({
           actual: status,
@@ -3444,6 +3448,10 @@ export default class Meeting extends StatelessWebexPlugin {
             receiveShare: this.mediaProperties.mediaDirection?.receiveShare,
           },
         });
+      } else {
+        LoggerProxy.logger.warn(
+          'Meeting:index#setUpLocusInfoSelfListener --> MEDIA_STATUS_CHANGE, statsAnalyzer is not available.'
+        );
       }
     });
 
@@ -7542,6 +7550,9 @@ export default class Meeting extends StatelessWebexPlugin {
           throw error;
         }
       }
+
+      LoggerProxy.logger.info(`${LOG_HEADER} media connected, finalizing...`);
+
       if (this.mediaProperties.hasLocalShareStream()) {
         await this.enqueueScreenShareFloorRequest();
       }

package/test/unit/spec/media/properties.ts

@@ -66,6 +66,21 @@ describe('MediaProperties', () => {
       assert.equal(numTransports, 0);
     });
 
+    it('handles time out in the case when getStats() is not resolving', async () => {
+      // Promise that never resolves
+      mockMC.getStats = new Promise(() => {});
+
+      const promise = mediaProperties.getCurrentConnectionInfo();
+
+      await clock.tickAsync(1000);
+
+      const {connectionType, selectedCandidatePairChanges, numTransports} = await promise;
+
+      assert.equal(connectionType, 'unknown');
+      assert.equal(selectedCandidatePairChanges, -1);
+      assert.equal(numTransports, 0);
+    });
+
     describe('selectedCandidatePairChanges and numTransports', () => {
       it('returns correct values when getStats() returns no transport stats at all', async () => {
         mockMC.getStats.resolves([{type: 'something', id: '1234'}]);