@webex/plugin-meetings 3.7.0 → 3.8.0-next.10

package/src/reachability/clusterReachability.ts

@@ -6,20 +6,7 @@ import {convertStunUrlToTurn, convertStunUrlToTurnTls} from './util';
 import EventsScope from '../common/events/events-scope';
 
 import {CONNECTION_STATE, Enum, ICE_GATHERING_STATE} from '../constants';
-
-// result for a specific transport protocol (like udp or tcp)
-export type TransportResult = {
-  result: 'reachable' | 'unreachable' | 'untested';
-  latencyInMilliseconds?: number; // amount of time it took to get the first ICE candidate
-  clientMediaIPs?: string[];
-};
-
-// reachability result for a specific media cluster
-export type ClusterReachabilityResult = {
-  udp: TransportResult;
-  tcp: TransportResult;
-  xtls: TransportResult;
-};
+import {ClusterReachabilityResult} from './reachability.types';
 
 // data for the Events.resultReady event
 export type ResultEventData = {
@@ -370,11 +357,14 @@ export class ClusterReachability extends EventsScope {
 
       this.startTimestamp = performance.now();
 
+      // Set up the state change listeners before triggering the ICE gathering
+      const gatherIceCandidatePromise = this.gatherIceCandidates();
+
       // not awaiting the next call on purpose, because we're not sending the offer anywhere and there won't be any answer
       // we just need to make this call to trigger the ICE gathering process
       this.pc.setLocalDescription(offer);
 
-      await this.gatherIceCandidates();
+      await gatherIceCandidatePromise;
     } catch (error) {
       LoggerProxy.logger.warn(`Reachability:ClusterReachability#start --> Error: `, error);
     }

package/src/reachability/index.ts

@@ -9,64 +9,31 @@ import {Defer} from '@webex/common';
 import LoggerProxy from '../common/logs/logger-proxy';
 import MeetingUtil from '../meeting/util';
 
-import {REACHABILITY} from '../constants';
+import {IP_VERSION, REACHABILITY} from '../constants';
 
 import ReachabilityRequest, {ClusterList} from './request';
+import {
+  ClusterReachabilityResult,
+  TransportResult,
+  ClientMediaPreferences,
+  ReachabilityMetrics,
+  ReachabilityReportV0,
+  ReachabilityReportV1,
+  ReachabilityResults,
+  ReachabilityResultsForBackend,
+  TransportResultForBackend,
+  GetClustersTrigger,
+} from './reachability.types';
 import {
   ClientMediaIpsUpdatedEventData,
   ClusterReachability,
-  ClusterReachabilityResult,
   Events,
   ResultEventData,
-  TransportResult,
 } from './clusterReachability';
 import EventsScope from '../common/events/events-scope';
 import BEHAVIORAL_METRICS from '../metrics/constants';
 import Metrics from '../metrics';
 
-export type ReachabilityMetrics = {
-  reachability_public_udp_success: number;
-  reachability_public_udp_failed: number;
-  reachability_public_tcp_success: number;
-  reachability_public_tcp_failed: number;
-  reachability_public_xtls_success: number;
-  reachability_public_xtls_failed: number;
-  reachability_vmn_udp_success: number;
-  reachability_vmn_udp_failed: number;
-  reachability_vmn_tcp_success: number;
-  reachability_vmn_tcp_failed: number;
-  reachability_vmn_xtls_success: number;
-  reachability_vmn_xtls_failed: number;
-};
-
-/**
- * This is the type that matches what backend expects us to send to them. It is a bit weird, because
- * it uses strings instead of booleans and numbers, but that's what they require.
- */
-export type TransportResultForBackend = {
-  reachable?: 'true' | 'false';
-  latencyInMilliseconds?: string;
-  clientMediaIPs?: string[];
-  untested?: 'true';
-};
-
-export type ReachabilityResultForBackend = {
-  udp: TransportResultForBackend;
-  tcp: TransportResultForBackend;
-  xtls: TransportResultForBackend;
-};
-
-// this is the type that is required by the backend when we send them reachability results
-export type ReachabilityResultsForBackend = Record<string, ReachabilityResultForBackend>;
-
-// this is the type used by Reachability class internally and stored in local storage
-export type ReachabilityResults = Record<
-  string,
-  ClusterReachabilityResult & {
-    isVideoMesh?: boolean;
-  }
->;
-
 // timeouts in seconds
 const DEFAULT_TIMEOUT = 3;
 const VIDEO_MESH_TIMEOUT = 1;
@@ -84,6 +51,9 @@ export default class Reachability extends EventsScope {
     [key: string]: ClusterReachability;
   };
 
+  minRequiredClusters?: number;
+  orpheusApiVersion?: number;
+
   reachabilityDefer?: Defer;
 
   vmnTimer?: ReturnType<typeof setTimeout>;
@@ -92,6 +62,8 @@ export default class Reachability extends EventsScope {
 
   expectedResultsCount = {videoMesh: {udp: 0}, public: {udp: 0, tcp: 0, xtls: 0}};
   resultsCount = {videoMesh: {udp: 0}, public: {udp: 0, tcp: 0, xtls: 0}};
+  startTime = undefined;
+  totalDuration = undefined;
 
   protected lastTrigger?: string;
 
@@ -118,14 +90,35 @@ export default class Reachability extends EventsScope {
 
   /**
    * Fetches the list of media clusters from the backend
+   * @param {string} trigger - explains the reason for starting reachability, used by Orpheus
+   * @param {Object} previousReport - last reachability report
    * @param {boolean} isRetry
    * @private
    * @returns {Promise<{clusters: ClusterList, joinCookie: any}>}
    */
-  async getClusters(isRetry = false): Promise<{clusters: ClusterList; joinCookie: any}> {
+  async getClusters(
+    trigger: GetClustersTrigger,
+    previousReport?: any,
+    isRetry = false
+  ): Promise<{
+    clusters: ClusterList;
+    joinCookie: any;
+  }> {
     try {
-      const {clusters, joinCookie} = await this.reachabilityRequest.getClusters(
-        MeetingUtil.getIpVersion(this.webex)
+      const {clusters, joinCookie, discoveryOptions} = await this.reachabilityRequest.getClusters(
+        trigger,
+        MeetingUtil.getIpVersion(this.webex),
+        previousReport
+      );
+
+      this.minRequiredClusters = discoveryOptions?.['early-call-min-clusters'];
+      this.orpheusApiVersion = discoveryOptions?.['report-version'];
+
+      // @ts-ignore
+      await this.webex.boundedStorage.put(
+        this.namespace,
+        REACHABILITY.localStorageJoinCookie,
+        JSON.stringify(joinCookie)
       );
 
       return {clusters, joinCookie};
@@ -138,7 +131,7 @@ export default class Reachability extends EventsScope {
         `Reachability:index#getClusters --> Failed with error: ${error}, retrying...`
       );
 
-      return this.getClusters(true);
+      return this.getClusters(trigger, previousReport, true);
     }
   }
 
@@ -150,6 +143,10 @@ export default class Reachability extends EventsScope {
    * @memberof Reachability
    */
   public async gatherReachability(trigger: string): Promise<ReachabilityResults> {
+    // @ts-ignore
+    if (!this.webex.config.meetings.enableReachabilityChecks) {
+      throw new Error('enableReachabilityChecks is disabled in config');
+    }
     // Fetch clusters and measure latency
     try {
       this.lastTrigger = trigger;
@@ -159,14 +156,7 @@ export default class Reachability extends EventsScope {
       // @ts-ignore
       this.webex.internal.device.ipNetworkDetector.detect(true);
 
-      const {clusters, joinCookie} = await this.getClusters();
-
-      // @ts-ignore
-      await this.webex.boundedStorage.put(
-        this.namespace,
-        REACHABILITY.localStorageJoinCookie,
-        JSON.stringify(joinCookie)
-      );
+      const {clusters} = await this.getClusters('startup');
 
       this.reachabilityDefer = new Defer();
 
@@ -181,6 +171,124 @@ export default class Reachability extends EventsScope {
     }
   }
 
+  /**
+   * Gets the last join cookie we got from Orpheus
+   *
+   * @returns {Promise<Object>} join cookie
+   */
+  async getJoinCookie() {
+    // @ts-ignore
+    const joinCookieRaw = await this.webex.boundedStorage
+      .get(REACHABILITY.namespace, REACHABILITY.localStorageJoinCookie)
+      .catch(() => {});
+
+    let joinCookie;
+
+    if (joinCookieRaw) {
+      try {
+        joinCookie = JSON.parse(joinCookieRaw);
+      } catch (e) {
+        LoggerProxy.logger.error(
+          `MeetingRequest#constructor --> Error in parsing join cookie data: ${e}`
+        );
+      }
+    }
+
+    return joinCookie;
+  }
+
+  /**
+   * Returns the reachability report that needs to be attached to the ROAP messages
+   * that we send to the backend.
+   *
+   * @returns {Promise<Object>}
+   */
+  async getReachabilityReport(): Promise<
+    | {
+        joinCookie: any;
+        reachability?: ReachabilityReportV1;
+      }
+    | {
+        reachability: ReachabilityReportV0;
+      }
+  > {
+    const reachabilityResult = await this.getReachabilityResults();
+    const joinCookie = await this.getJoinCookie();
+
+    // Orpheus API version 0
+    if (!this.orpheusApiVersion) {
+      return {
+        reachability: reachabilityResult,
+      };
+    }
+
+    // Orpheus API version 1
+    return {
+      reachability: {
+        version: 1,
+        result: {
+          usedDiscoveryOptions: {
+            'early-call-min-clusters': this.minRequiredClusters,
+          },
+          metrics: {
+            'total-duration-ms': this.totalDuration,
+          },
+          tests: reachabilityResult,
+        },
+      },
+      joinCookie,
+    };
+  }
+
+  /**
+   * This method is called when we don't succeed in reaching the minimum number of clusters
+   * required by Orpheus. It sends the results to Orpheus and gets a new list that it tries to reach again.
+   * @returns {Promise<ReachabilityResults>} reachability results
+   * @public
+   * @memberof Reachability
+   */
+  public async gatherReachabilityFallback(): Promise<void> {
+    try {
+      const reachabilityReport = await this.getReachabilityReport();
+
+      const {clusters} = await this.getClusters('early-call/no-min-reached', reachabilityReport);
+
+      // stop all previous reachability checks that might still be going on in the background
+      this.abortCurrentChecks();
+
+      // Perform Reachability Check
+      await this.performReachabilityChecks(clusters);
+    } catch (error) {
+      LoggerProxy.logger.error(`Reachability:index#gatherReachabilityFallback --> Error:`, error);
+    }
+  }
+
+  /**
+   * Stops all reachability checks that are in progress
+   * @public
+   * @memberof Reachability
+   * @returns {void}
+   */
+  public stopReachability() {
+    // overallTimer is always there only if there is reachability in progress
+    if (this.overallTimer) {
+      LoggerProxy.logger.log(
+        'Reachability:index#stopReachability --> stopping reachability checks'
+      );
+      this.abortCurrentChecks();
+      this.emit(
+        {
+          file: 'reachability',
+          function: 'stopReachability',
+        },
+        'reachability:stopped',
+        {}
+      );
+      this.sendMetric(true);
+      this.resolveReachabilityPromise();
+    }
+  }
+
   /**
    * Returns statistics about last reachability results. The returned value is an object
    * with a flat list of properties so that it can be easily sent with metrics
@@ -304,7 +412,7 @@ export default class Reachability extends EventsScope {
     } catch (e) {
       // empty storage, that's ok
       LoggerProxy.logger.warn(
-        'Roap:request#attachReachabilityData --> Error parsing reachability data: ',
+        'Reachability:index#getReachabilityResults --> Error parsing reachability data: ',
         e
       );
     }
@@ -336,7 +444,7 @@ export default class Reachability extends EventsScope {
         );
       } catch (e) {
         LoggerProxy.logger.error(
-          `Roap:request#attachReachabilityData --> Error in parsing reachability data: ${e}`
+          `Reachability:index#isAnyPublicClusterReachable --> Error in parsing reachability data: ${e}`
         );
       }
     }
@@ -393,7 +501,7 @@ export default class Reachability extends EventsScope {
         );
       } catch (e) {
         LoggerProxy.logger.error(
-          `Roap:request#attachReachabilityData --> Error in parsing reachability data: ${e}`
+          `Reachability:index#isWebexMediaBackendUnreachable --> Error in parsing reachability data: ${e}`
         );
       }
     }
@@ -427,6 +535,30 @@ export default class Reachability extends EventsScope {
     return unreachableList;
   }
 
+  /**
+   * Gets the number of reachable clusters from last run reachability check
+   * @returns {number} reachable clusters count
+   * @private
+   * @memberof Reachability
+   */
+  private getNumberOfReachableClusters(): number {
+    let count = 0;
+
+    Object.entries(this.clusterReachability).forEach(([key, clusterReachability]) => {
+      const result = clusterReachability.getResult();
+
+      if (
+        result.udp.result === 'reachable' ||
+        result.tcp.result === 'reachable' ||
+        result.xtls.result === 'reachable'
+      ) {
+        count += 1;
+      }
+    });
+
+    return count;
+  }
+
   /**
    * Make a log of unreachable clusters.
    * @returns {undefined}
@@ -465,18 +597,27 @@ export default class Reachability extends EventsScope {
 
   /**
    * Resolves the promise returned by gatherReachability() method
+   * @param {boolean} checkMinRequiredClusters - if true, it will check if we have reached the minimum required clusters and do a fallback if needed
    * @returns {void}
    */
-  private resolveReachabilityPromise() {
-    if (this.vmnTimer) {
-      clearTimeout(this.vmnTimer);
-    }
-    if (this.publicCloudTimer) {
-      clearTimeout(this.publicCloudTimer);
-    }
+  private resolveReachabilityPromise(checkMinRequiredClusters = true) {
+    this.totalDuration = performance.now() - this.startTime;
+
+    this.clearTimer('vmnTimer');
+    this.clearTimer('publicCloudTimer');
 
     this.logUnreachableClusters();
     this.reachabilityDefer?.resolve();
+
+    if (checkMinRequiredClusters) {
+      const numReachableClusters = this.getNumberOfReachableClusters();
+      if (this.minRequiredClusters && numReachableClusters < this.minRequiredClusters) {
+        LoggerProxy.logger.log(
+          `Reachability:index#resolveReachabilityPromise --> minRequiredClusters not reached (${numReachableClusters} < ${this.minRequiredClusters}), doing reachability fallback`
+        );
+        this.gatherReachabilityFallback();
+      }
+    }
   }
 
   /**
@@ -526,9 +667,10 @@ export default class Reachability extends EventsScope {
   /**
    * Sends a metric with all the statistics about how long reachability took
    *
+   * @param {boolean} aborted true if the reachability checks were aborted
    * @returns {void}
    */
-  protected async sendMetric() {
+  protected async sendMetric(aborted = false) {
     const results = [];
 
     Object.values(this.clusterReachability).forEach((clusterReachability) => {
@@ -539,6 +681,7 @@ export default class Reachability extends EventsScope {
     });
 
     const stats = {
+      aborted,
       vmn: {
         udp: this.getStatistics(results, 'udp', true),
       },
@@ -591,6 +734,8 @@ export default class Reachability extends EventsScope {
         `Reachability:index#startTimers --> Reachability checks timed out (${DEFAULT_TIMEOUT}s)`
       );
 
+      // check against minimum required clusters, do a new call if we don't have enough
+
       // resolve the promise, so that the client won't be blocked waiting on meetings.register() for too long
       this.resolveReachabilityPromise();
     }, DEFAULT_TIMEOUT * 1000);
@@ -646,6 +791,32 @@ export default class Reachability extends EventsScope {
     this.resultsCount.public.xtls = 0;
   }
 
+  /**
+   * Clears the timer
+   *
+   * @param {string} timer name of the timer to clear
+   * @returns {void}
+   */
+  private clearTimer(timer: string) {
+    if (this[timer]) {
+      clearTimeout(this[timer]);
+      this[timer] = undefined;
+    }
+  }
+
+  /**
+   * Aborts current checks that are in progress
+   *
+   * @returns {void}
+   */
+  private abortCurrentChecks() {
+    this.clearTimer('vmnTimer');
+    this.clearTimer('publicCloudTimer');
+    this.clearTimer('overallTimer');
+
+    this.abortClusterReachability();
+  }
+
   /**
    * Performs reachability checks for all clusters
    * @param {ClusterList} clusterList
@@ -656,9 +827,7 @@ export default class Reachability extends EventsScope {
 
     this.clusterReachability = {};
 
-    if (!clusterList || !Object.keys(clusterList).length) {
-      return;
-    }
+    this.startTime = performance.now();
 
     LoggerProxy.logger.log(
       `Reachability:index#performReachabilityChecks --> doing UDP${
@@ -671,7 +840,6 @@ export default class Reachability extends EventsScope {
     );
 
     this.resetResultCounters();
-    this.startTimers();
 
     // sanitize the urls in the clusterList
     Object.keys(clusterList).forEach((key) => {
@@ -721,6 +889,24 @@ export default class Reachability extends EventsScope {
     // save the initialized results (in case we don't get any "resultReady" events at all)
     await this.storeResults(results);
 
+    if (!clusterList || !Object.keys(clusterList).length) {
+      // nothing to do, finish immediately
+      this.resolveReachabilityPromise(false);
+
+      this.emit(
+        {
+          file: 'reachability',
+          function: 'performReachabilityChecks',
+        },
+        'reachability:done',
+        {}
+      );
+
+      return;
+    }
+
+    this.startTimers();
+
     // now start the reachability on all the clusters
     Object.keys(clusterList).forEach((key) => {
       const cluster = clusterList[key];
@@ -753,8 +939,7 @@ export default class Reachability extends EventsScope {
         await this.storeResults(results);
 
         if (areAllResultsReady) {
-          clearTimeout(this.overallTimer);
-          this.overallTimer = undefined;
+          this.clearTimer('overallTimer');
           this.emit(
             {
               file: 'reachability',
@@ -785,4 +970,59 @@ export default class Reachability extends EventsScope {
       this.clusterReachability[key].start(); // not awaiting on purpose
     });
   }
+
+  /**
+   * Returns the clientMediaPreferences object that needs to be sent to the backend
+   * when joining a meeting
+   *
+   * @param {boolean} isMultistream
+   * @param {IP_VERSION} ipver
+   * @returns {Object}
+   */
+  async getClientMediaPreferences(
+    isMultistream: boolean,
+    ipver?: IP_VERSION
+  ): Promise<ClientMediaPreferences> {
+    // if 0 or undefined, we assume version 0 and don't send any reachability in clientMediaPreferences
+    if (!this.orpheusApiVersion) {
+      return {
+        ipver,
+        joinCookie: await this.getJoinCookie(),
+        preferTranscoding: !isMultistream,
+      };
+    }
+
+    // must be version 1
+
+    // for version 1, the reachability report goes into clientMediaPreferences (and it contains joinCookie)
+    const reachabilityReport = (await this.getReachabilityReport()) as {
+      joinCookie: any;
+      reachability?: ReachabilityReportV1;
+    };
+
+    return {
+      ipver,
+      preferTranscoding: !isMultistream,
+      ...reachabilityReport,
+    };
+  }
+
+  /**
+   * Returns the reachability report that needs to be attached to the ROAP messages
+   * that we send to the backend.
+   * It may return undefined, if reachability is not needed to be attached to ROAP messages (that's the case for v1 or Orpheus API)
+   *
+   * @returns {Promise<ReachabilityReportV0>} object that needs to be attached to Roap messages
+   */
+  async getReachabilityReportToAttachToRoap(): Promise<ReachabilityReportV0 | undefined> {
+    // version 0
+    if (!this.orpheusApiVersion) {
+      return this.getReachabilityResults();
+    }
+
+    // version 1
+
+    // for version 1 we don't attach anything to Roap messages, reachability report is sent inside clientMediaPreferences
+    return undefined;
+  }
 }

package/src/reachability/reachability.types.ts

@@ -0,0 +1,85 @@
+import {IP_VERSION} from '../constants';
+
+// result for a specific transport protocol (like udp or tcp)
+export type TransportResult = {
+  result: 'reachable' | 'unreachable' | 'untested';
+  latencyInMilliseconds?: number; // amount of time it took to get the first ICE candidate
+  clientMediaIPs?: string[];
+};
+
+// reachability result for a specific media cluster
+export type ClusterReachabilityResult = {
+  udp: TransportResult;
+  tcp: TransportResult;
+  xtls: TransportResult;
+};
+
+export type ReachabilityMetrics = {
+  reachability_public_udp_success: number;
+  reachability_public_udp_failed: number;
+  reachability_public_tcp_success: number;
+  reachability_public_tcp_failed: number;
+  reachability_public_xtls_success: number;
+  reachability_public_xtls_failed: number;
+  reachability_vmn_udp_success: number;
+  reachability_vmn_udp_failed: number;
+  reachability_vmn_tcp_success: number;
+  reachability_vmn_tcp_failed: number;
+  reachability_vmn_xtls_success: number;
+  reachability_vmn_xtls_failed: number;
+};
+
+/**
+ * This is the type that matches what backend expects us to send to them. It is a bit weird, because
+ * it uses strings instead of booleans and numbers, but that's what they require.
+ */
+export type TransportResultForBackend = {
+  reachable?: 'true' | 'false';
+  latencyInMilliseconds?: string;
+  clientMediaIPs?: string[];
+  untested?: 'true';
+};
+
+export type ReachabilityResultForBackend = {
+  udp: TransportResultForBackend;
+  tcp: TransportResultForBackend;
+  xtls: TransportResultForBackend;
+};
+
+// this is the type that is required by the backend when we send them reachability results
+export type ReachabilityResultsForBackend = Record<string, ReachabilityResultForBackend>;
+
+// this is the type used by Reachability class internally and stored in local storage
+export type ReachabilityResults = Record<
+  string,
+  ClusterReachabilityResult & {
+    isVideoMesh?: boolean;
+  }
+>;
+
+export type ReachabilityReportV0 = ReachabilityResultsForBackend;
+
+export type ReachabilityReportV1 = {
+  version: 1;
+  result: {
+    usedDiscoveryOptions: {
+      'early-call-min-clusters': number;
+      // there are more options, but we don't support them yet
+    };
+    metrics: {
+      'total-duration-ms': number;
+      // there are more metrics, but we don't support them yet
+    };
+    tests: Record<string, ReachabilityResultForBackend>;
+  };
+};
+
+export interface ClientMediaPreferences {
+  ipver: IP_VERSION;
+  joinCookie: any;
+  preferTranscoding: boolean;
+  reachability?: ReachabilityReportV1; // only present when using Orpheus API version 1
+}
+
+/* Orpheus API supports more triggers, but we don't use them yet */
+export type GetClustersTrigger = 'startup' | 'early-call/no-min-reached';