@webex/plugin-meetings 2.36.1 → 2.37.1

package/src/metrics/index.ts

@@ -13,8 +13,13 @@ import {MEETING_ERRORS} from '../constants';
 import BrowserDetection from '../common/browser-detection';
 
 import {
-  error, eventType, errorCodes as ERROR_CODE, OS_NAME, UNKNOWN, CLIENT_NAME,
-  mediaType
+  error,
+  eventType,
+  errorCodes as ERROR_CODE,
+  OS_NAME,
+  UNKNOWN,
+  CLIENT_NAME,
+  mediaType,
 } from './config';
 
 const OSMap = {
@@ -24,15 +29,10 @@ const OSMap = {
   Windows: OS_NAME.WINDOWS,
   iOS: OS_NAME.IOS,
   Android: OS_NAME.ANDROID,
-  Linux: OS_NAME.LINUX
+  Linux: OS_NAME.LINUX,
 };
 
-const {
-  getOSName,
-  getOSVersion,
-  getBrowserName,
-  getBrowserVersion
-} = BrowserDetection();
+const {getOSName, getOSVersion, getBrowserName, getBrowserVersion} = BrowserDetection();
 
 // Apply a CIDR /28 format to the IPV4 and /96 to the IPV6 addresses
 // For reference : https://en.wikipedia.org/wiki/Classless_Inter-Domain_Routing
@@ -85,33 +85,33 @@ class Metrics {
   webex: any;
 
   /**
-     * Create Metrics Object
-     * @constructor
-     * @public
-     * @memberof Meetings
-     */
+   * Create Metrics Object
+   * @constructor
+   * @public
+   * @memberof Meetings
+   */
   constructor() {
     if (!Metrics.instance) {
-    /**
-     * @instance
-     * @type {Array}
-     * @private
-     * @memberof Metrics
-     */
+      /**
+       * @instance
+       * @type {Array}
+       * @private
+       * @memberof Metrics
+       */
       this._events = [];
       /**
-     * @instance
-     * @type {MeetingCollection}
-     * @private
-     * @memberof Metrics
-     */
+       * @instance
+       * @type {MeetingCollection}
+       * @private
+       * @memberof Metrics
+       */
       this.meetingCollection = null;
       /**
-     * @instance
-     * @type {MeetingCollection}
-     * @private
-     * @memberof Metrics
-     */
+       * @instance
+       * @type {MeetingCollection}
+       * @private
+       * @memberof Metrics
+       */
       this.keys = Object.values(eventType);
       /**
        * @instance
@@ -122,6 +122,7 @@ class Metrics {
       Metrics.instance = this;
     }
 
+    // eslint-disable-next-line no-constructor-return
     return Metrics.instance;
   }
 
@@ -147,12 +148,14 @@ class Metrics {
    * @param {object} options.event
    * @returns {object} null
    */
-  postEvent(options: { meeting?: any; meetingId?: string; data?: object; event?: any } | any) {
+  postEvent(options: {meeting?: any; meetingId?: string; data?: object; event?: any} | any) {
     const {meetingId, data = {}, event} = options;
     let {meeting} = options;
 
     if (this.keys.indexOf(event) === -1) {
-      LoggerProxy.logger.error(`Metrics:index#postEvent --> Event ${event} doesn't exist in dictionary`);
+      LoggerProxy.logger.error(
+        `Metrics:index#postEvent --> Event ${event} doesn't exist in dictionary`
+      );
     }
 
     if (!meeting && meetingId) {
@@ -169,25 +172,25 @@ class Metrics {
       if (event === eventType.MEDIA_QUALITY) {
         data.event = event;
         meeting.sendMediaQualityAnalyzerMetrics(data);
-      }
-      else {
+      } else {
         meeting.callEvents.push(event);
         data.event = event;
         meeting.sendCallAnalyzerMetrics(data);
       }
-    }
-
-    else {
-      LoggerProxy.logger.info(`Metrics:index#postEvent --> Event received for meetingId:${meetingId}, but meeting not found in collection.`);
+    } else {
+      LoggerProxy.logger.info(
+        `Metrics:index#postEvent --> Event received for meetingId:${meetingId}, but meeting not found in collection.`
+      );
     }
   }
 
   /**
    *  Docs for Call analyzer metrics
-  *   https://sqbu-github.cisco.com/WebExSquared/call-analyzer/wiki
-  *   https://sqbu-github.cisco.com/WebExSquared/event-dictionary/blob/master/diagnostic-events.raml
- */
+   *   https://sqbu-github.cisco.com/WebExSquared/call-analyzer/wiki
+   *   https://sqbu-github.cisco.com/WebExSquared/event-dictionary/blob/master/diagnostic-events.raml
+   */
 
+  // eslint-disable-next-line @typescript-eslint/no-shadow
   initPayload(eventType, identifiers, options) {
     const payload: any = {
       eventId: uuid.v4(),
@@ -204,19 +207,19 @@ class Metrics {
           subClientType: options.subClientType,
           os: this.getOsName(),
           browser: getBrowserName(),
-          browserVersion: getBrowserVersion()
-        }
+          browserVersion: getBrowserVersion(),
+        },
       },
       originTime: {
-        triggered: new Date().toISOString()
+        triggered: new Date().toISOString(),
       },
       senderCountryCode: this.webex.meetings.geoHintInfo?.countryCode,
       event: {
         name: eventType,
         canProceed: true,
         identifiers,
-        eventData: {webClientDomain: window.location.hostname}
-      }
+        eventData: {webClientDomain: window.location.hostname},
+      },
     };
 
     // TODO: more options should be checked and some of them should be mandatory in certain conditions
@@ -279,16 +282,19 @@ class Metrics {
    * @memberof Metrics
    */
   public initMediaPayload(
+    // eslint-disable-next-line @typescript-eslint/no-shadow
     eventType: string,
     identifiers: {
       correlationId: string;
       locusUrl: string;
       locusId: string;
     },
-    options: {
-      intervalData: object;
-      clientType: string;
-    } | any = {}
+    options:
+      | {
+          intervalData: object;
+          clientType: string;
+        }
+      | any = {}
   ) {
     const {audioSetupDelay, videoSetupDelay, joinTimes} = options;
 
@@ -309,11 +315,11 @@ class Metrics {
           osVersion: getOSVersion() || UNKNOWN,
           subClientType: options.subClientType,
           browser: getBrowserName(),
-          browserVersion: getBrowserVersion()
-        }
+          browserVersion: getBrowserVersion(),
+        },
       },
       originTime: {
-        triggered: new Date().toISOString()
+        triggered: new Date().toISOString(),
       },
       senderCountryCode: this.webex.meetings.geoHintInfo?.countryCode,
       event: {
@@ -323,16 +329,16 @@ class Metrics {
         intervals: [options.intervalData],
         joinTimes,
         eventData: {
-          webClientDomain: window.location.hostname
+          webClientDomain: window.location.hostname,
         },
         sourceMetadata: {
           applicationSoftwareType: CLIENT_NAME,
           applicationSoftwareVersion: this.webex.version,
           mediaEngineSoftwareType: getBrowserName() || 'browser',
           mediaEngineSoftwareVersion: getOSVersion() || UNKNOWN,
-          startTime: new Date().toISOString()
-        }
-      }
+          startTime: new Date().toISOString(),
+        },
+      },
     };
 
     return payload;
@@ -353,8 +359,7 @@ class Metrics {
 
     if (err && err.statusCode && err.statusCode >= 500) {
       errorCode = 1003;
-    }
-    else if (err && err.body && err.body.errorCode) {
+    } else if (err && err.body && err.body.errorCode) {
       // locus error codes: https://sqbu-github.cisco.com/WebExSquared/locus/blob/master/server/src/main/resources/locus-error-codes.properties
       switch (ERROR_CODE[err.body.errorCode]) {
         case MEETING_ERRORS.FREE_USER_MAX_PARTICIPANTS_EXCEEDED:
@@ -453,17 +458,13 @@ class Metrics {
         default:
           errorCode = 4008;
       }
-    }
-    else {
+    } else {
       errorCode = 4008;
     }
 
-    return this.generateErrorPayload(
-      errorCode, showToUser, error.name.LOCUS_RESPONSE, err
-    );
+    return this.generateErrorPayload(errorCode, showToUser, error.name.LOCUS_RESPONSE, err);
   }
 
-
   generateErrorPayload(errorCode, shownToUser, name, err) {
     if (error.errors[errorCode]) {
       const errorPayload: any = {
@@ -472,7 +473,7 @@ class Metrics {
         errorDescription: error.errors[errorCode][0],
         errorCode,
         fatal: !includes(error.notFatalErrorList, errorCode),
-        name: name || error.name.OTHER
+        name: name || error.name.OTHER,
       };
 
       if (err && err.body) {
@@ -499,8 +500,14 @@ class Metrics {
     let browserInfo;
     const clientInfo = util.format('client=%s', `${this.webex.meetings?.metrics?.clientName}`);
 
-    if (['chrome', 'firefox', 'msie', 'msedge', 'safari'].indexOf(getBrowserName().toLowerCase()) !== -1) {
-      browserInfo = util.format('browser=%s', `${getBrowserName().toLowerCase()}/${getBrowserVersion().split('.')[0]}`);
+    if (
+      ['chrome', 'firefox', 'msie', 'msedge', 'safari'].indexOf(getBrowserName().toLowerCase()) !==
+      -1
+    ) {
+      browserInfo = util.format(
+        'browser=%s',
+        `${getBrowserName().toLowerCase()}/${getBrowserVersion().split('.')[0]}`
+      );
     }
     const osInfo = util.format('os=%s', `${getOSName()}/${getOSVersion().split('.')[0]}`);
 
@@ -508,12 +515,18 @@ class Metrics {
       userAgentOption = `(${browserInfo}`;
     }
     if (osInfo) {
-      userAgentOption = userAgentOption ? `${userAgentOption}; ${clientInfo}; ${osInfo}` : `${clientInfo}; (${osInfo}`;
+      userAgentOption = userAgentOption
+        ? `${userAgentOption}; ${clientInfo}; ${osInfo}`
+        : `${clientInfo}; (${osInfo}`;
     }
     if (userAgentOption) {
       userAgentOption += ')';
 
-      return util.format('webex-js-sdk/%s %s', `${process.env.NODE_ENV}-${this.webex.version}`, userAgentOption);
+      return util.format(
+        'webex-js-sdk/%s %s',
+        `${process.env.NODE_ENV}-${this.webex.version}`,
+        userAgentOption
+      );
     }
 
     return util.format('webex-js-sdk/%s', `${process.env.NODE_ENV}-${this.webex.version}`);
@@ -538,7 +551,7 @@ class Metrics {
     this.webex.internal.metrics.submitClientMetrics(metricName, {
       type: this.webex.config.metrics.type,
       fields: metricFields,
-      tags: metricTags
+      tags: metricTags,
     });
   }
 }

package/src/networkQualityMonitor/index.ts

@@ -1,16 +1,15 @@
 import EventsScope from '../common/events/events-scope';
 import {EVENT_TRIGGERS, STATS} from '../constants';
 
-
 /**
-  * Meeting - network quality event
-  * Emitted on each interval of retrieving stats Analyzer data
-  * @event network:quality
-  * @type {Object}
-  * @property {string} mediaType {video|audio}
-  * @property {number} networkQualityScore - value determined in determineUplinkNetworkQuality
-  * @memberof NetworkQualityMonitor
-  */
+ * Meeting - network quality event
+ * Emitted on each interval of retrieving stats Analyzer data
+ * @event network:quality
+ * @type {Object}
+ * @property {string} mediaType {video|audio}
+ * @property {number} networkQualityScore - value determined in determineUplinkNetworkQuality
+ * @memberof NetworkQualityMonitor
+ */
 /**
  * NetworkQualityMonitor class that will emit events based on detected quality
  *
@@ -42,19 +41,19 @@ export default class NetworkQualityMonitor extends EventsScope {
     this.indicatorTypes = Object.freeze({
       PACKETLOSS: 'packetLoss',
       LATENCY: 'latency',
-      JITTER: 'jitter'
+      JITTER: 'jitter',
     });
     this.frequencyTypes = Object.freeze({
       UPLINK: 'uplink',
-      DOWNLINK: 'downlink'
+      DOWNLINK: 'downlink',
     });
     this.networkQualityScore = 1;
     this.networkQualityStatus = {
       [this.frequencyTypes.UPLINK]: {
         [STATS.VIDEO_CORRELATE]: {},
         [STATS.AUDIO_CORRELATE]: {},
-        [STATS.SHARE_CORRELATE]: {}
-      }
+        [STATS.SHARE_CORRELATE]: {},
+      },
     };
     this.mediaType = null;
   }
@@ -69,12 +68,12 @@ export default class NetworkQualityMonitor extends EventsScope {
     this.emit(
       {
         file: 'networkQualityMonitor',
-        function: 'emitNetworkQuality'
+        function: 'emitNetworkQuality',
       },
       EVENT_TRIGGERS.NETWORK_QUALITY,
       {
         mediaType: this.mediaType,
-        networkQualityScore: this.networkQualityScore
+        networkQualityScore: this.networkQualityScore,
       }
     );
   }
@@ -92,7 +91,6 @@ export default class NetworkQualityMonitor extends EventsScope {
     this.mediaType = null;
   }
 
-
   /**
    * filter data to determine uplink network quality, invoked on same interval as stats analyzer remote-inbout-rtp
    * @param {Object} configObj
@@ -127,8 +125,7 @@ export default class NetworkQualityMonitor extends EventsScope {
      * @returns {boolean}
      */
     const determinePacketLoss = () => {
-      if (currentPacketLossRatio >
-        this.config.videoPacketLossRatioThreshold) {
+      if (currentPacketLossRatio > this.config.videoPacketLossRatioThreshold) {
         this.networkQualityScore = 0;
 
         return false;
@@ -172,7 +169,8 @@ export default class NetworkQualityMonitor extends EventsScope {
      * @param {(number|undefined)} value
      * @returns {(number|null)}
      */
-    const determineIfUndefined = (value: number | undefined) => (typeof value === 'undefined' ? null : value);
+    const determineIfUndefined = (value: number | undefined) =>
+      typeof value === 'undefined' ? null : value;
 
     /**
      * Values for some browsers specifically Safari will be undefined we explicitly set to null
@@ -182,20 +180,19 @@ export default class NetworkQualityMonitor extends EventsScope {
     // PACKET LOSS
     this.networkQualityStatus[UPLINK][mediaType][PACKETLOSS] = {
       acceptable: determinePacketLoss(),
-      value: determineIfUndefined(currentPacketLossRatio)
+      value: determineIfUndefined(currentPacketLossRatio),
     };
 
-
     // LATENCY measured in Round trip time
     this.networkQualityStatus[UPLINK][mediaType][LATENCY] = {
       acceptable: determineLatency(),
-      value: determineIfUndefined(remoteRtpResults.roundTripTime)
+      value: determineIfUndefined(remoteRtpResults.roundTripTime),
     };
 
     // JITTER
     this.networkQualityStatus[UPLINK][mediaType][JITTER] = {
       acceptable: deterMineJitter(),
-      value: determineIfUndefined(remoteRtpResults.jitter)
+      value: determineIfUndefined(remoteRtpResults.jitter),
     };
 
     this.updateNetworkQualityStatus();