@webex/internal-plugin-metrics 3.7.0 → 3.8.0-next.10

package/dist/types/business-metrics.d.ts

@@ -1,51 +1,33 @@
 import GenericMetrics from './generic-metrics';
 import { EventPayload, Table } from './metrics.types';
 /**
- * @description Util class to handle Buisness Metrics
+ * @description Util class to handle Business Metrics
  * @export
  * @class BusinessMetrics
  */
 export default class BusinessMetrics extends GenericMetrics {
     /**
-     * unfortunately, the pinot team does not allow changes to the schema of wbxapp_callend_metrics
-     * so we have to shim this layer specifically for this
-     * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table+wbxapp_callend_metrics
-     * @param {EventPayload} payload payload of the metric
-     * @returns {Promise<any>}
-     */
-    private submitCallEndEvent;
-    /**
-     * Submit a buisness metric to our metrics endpoint, going to the default business_ucf table
-     * all event payload keys are converted into a hex string value
-     * unfortunately, the pinot team does not allow changes to the schema of business_metrics
-     * so we have to shim this layer specifically for this
-     * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table%3A+business_metrics
-     * @param {string} name of the metric
-     * @param {EventPayload} payload payload of the metric
-     * @returns {Promise<any>}
-     */
-    private submitBusinessMetricsEvent;
-    /**
-     * Submit a buisness metric to our metrics endpoint, going to the default business_ucf table
-     * all event payload keys are converted into a hex string value
-     * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Business+metrics++-%3E+ROMA
+     * Build the metric event to submit.
      * @param {string} name of the metric
-     * @param {EventPayload} user payload of the metric
-     * @returns {Promise<any>}
+     * @param {EventPayload} payload user payload of the metric
+     * @param {EventPayload} metadata to include outside of eventPayload.value
+     * @returns {MetricEvent} The constructed metric event
      */
-    private submitDefaultEvent;
+    private buildEvent;
     /**
-     * Submit a buisness metric to our metrics endpoint.
+     * Submit a business metric to our metrics endpoint.
      * routes to the correct table with the correct schema payload by table
      * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Business+metrics++-%3E+ROMA
      * @param {string} name of the metric, ignored if going to wbxapp_callend_metrics
      * @param {EventPayload} payload user payload of the metric
      * @param {Table} table optional - to submit the metric to and adapt the sent schema
+     * @param {EventPayload} metadata optional - to include outside of eventPayload.value
      * @returns {Promise<any>}
      */
-    submitBusinessEvent({ name, payload, table, }: {
+    submitBusinessEvent({ name, payload, table, metadata, }: {
         name: string;
         payload: EventPayload;
         table?: Table;
+        metadata?: EventPayload;
     }): Promise<void>;
 }

package/dist/types/call-diagnostic/call-diagnostic-metrics.d.ts

@@ -29,6 +29,7 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
     private logger;
     private hasLoggedBrowserSerial;
     private device;
+    private delayedClientEvents;
     validator: (options: {
         type: 'mqe' | 'ce';
         event: Event;
@@ -196,6 +197,7 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
         provisionalCorrelationId?: string;
         roomId?: string;
         sessionCorrelationId?: string;
+        sharedMediaId?: string;
         sipCallId?: string;
         sipSessionId?: {
             local?: string;
@@ -223,6 +225,8 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
         webexSiteUuid?: string;
         webexUserId?: number;
         webexWebDomain?: string;
+        recordingId?: string;
+        clientCorrelationId?: string;
         correlationId: string;
         additionalProperties?: false;
     } | {
@@ -276,6 +280,7 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
         provisionalCorrelationId?: string;
         roomId?: string;
         sessionCorrelationId?: string;
+        sharedMediaId?: string;
         sipCallId?: string;
         sipSessionId?: {
             local?: string;
@@ -303,6 +308,8 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
         webexSiteUuid?: string;
         webexUserId?: number;
         webexWebDomain?: string;
+        recordingId?: string;
+        clientCorrelationId?: string;
         correlationId: string;
         additionalProperties?: false;
     };
@@ -401,13 +408,19 @@ export default class CallDiagnosticMetrics extends StatelessWebexPlugin {
      * @param arg.event - event key
      * @param arg.payload - additional payload to be merged with default payload
      * @param arg.options - payload
+     * @param arg.delaySubmitEvent - a boolean value indicating whether to delay the submission of client events.
      * @throws
      */
-    submitClientEvent({ name, payload, options, }: {
+    submitClientEvent({ name, payload, options, delaySubmitEvent, }: {
         name: ClientEvent['name'];
         payload?: ClientEventPayload;
         options?: SubmitClientEventOptions;
+        delaySubmitEvent?: boolean;
     }): Promise<any>;
+    /**
+     * Submit Delayed Client Event CA events. Clears delayedClientEvents array after submission.
+     */
+    submitDelayedClientEvents(): Promise<any[]> | Promise<void>;
     /**
      * Prepare the event and send the request to metrics-a service.
      * @param event

package/dist/types/call-diagnostic/config.d.ts

@@ -11,6 +11,7 @@ export declare const DTLS_HANDSHAKE_FAILED_CLIENT_CODE = 2008;
 export declare const ICE_FAILED_WITH_TURN_TLS_CLIENT_CODE = 2010;
 export declare const ICE_FAILED_WITHOUT_TURN_TLS_CLIENT_CODE = 2009;
 export declare const ICE_AND_REACHABILITY_FAILED_CLIENT_CODE = 2011;
+export declare const MULTISTREAM_NOT_AVAILABLE_CLIENT_CODE = 2012;
 export declare const WBX_APP_API_URL = "wbxappapi";
 export declare const WEBEX_SUB_SERVICE_TYPES: Record<string, ClientSubServiceType>;
 export declare const BROWSER_MEDIA_ERROR_NAME_TO_CLIENT_ERROR_CODES_MAP: {
@@ -94,6 +95,7 @@ export declare const ERROR_DESCRIPTIONS: {
     ICE_FAILED_WITHOUT_TURN_TLS: string;
     ICE_FAILED_WITH_TURN_TLS: string;
     ICE_AND_REACHABILITY_FAILED: string;
+    MULTISTREAM_NOT_AVAILABLE: string;
     SDP_OFFER_CREATION_ERROR: string;
     SDP_OFFER_CREATION_ERROR_MISSING_CODEC: string;
     WDM_RESTRICTED_REGION: string;

package/dist/types/generic-metrics.d.ts

@@ -17,7 +17,7 @@ export default abstract class GenericMetrics extends StatelessWebexPlugin {
      */
     constructor(...args: any[]);
     /**
-     * Submit a buisness metric to our metrics endpoint.
+     * Submit a business metric to our metrics endpoint.
      * @param {string} kind of metric for logging
      * @param {string} name of the metric
      * @param {object} event
@@ -44,7 +44,7 @@ export default abstract class GenericMetrics extends StatelessWebexPlugin {
      */
     protected getBrowserDetails(): object;
     /**
-     * Returns true once we have the deviceId we need to submit behavioral/operational/buisness events
+     * Returns true once we have the deviceId we need to submit behavioral/operational/business events
      * @returns {boolean}
      */
     isReadyToSubmitEvents(): boolean;

package/dist/types/index.d.ts

@@ -4,7 +4,7 @@
 import config from './config';
 import NewMetrics from './new-metrics';
 import * as Utils from './utils';
-import { ClientEvent, ClientEventLeaveReason, SubmitBehavioralEvent, SubmitClientEvent, SubmitInternalEvent, SubmitOperationalEvent, SubmitMQE, PreComputedLatencies } from './metrics.types';
+import { ClientEvent, ClientEventLeaveReason, SubmitBehavioralEvent, SubmitClientEvent, SubmitInternalEvent, SubmitOperationalEvent, SubmitBusinessEvent, SubmitMQE, PreComputedLatencies } from './metrics.types';
 import * as CALL_DIAGNOSTIC_CONFIG from './call-diagnostic/config';
 import * as CallDiagnosticUtils from './call-diagnostic/call-diagnostic-metrics.util';
 import CallDiagnosticMetrics from './call-diagnostic/call-diagnostic-metrics';
@@ -15,4 +15,4 @@ import BusinessMetrics from './business-metrics';
 import RtcMetrics from './rtcMetrics';
 export { default, getOSNameInternal } from './metrics';
 export { config, CALL_DIAGNOSTIC_CONFIG, NewMetrics, Utils, CallDiagnosticUtils, CallDiagnosticLatencies, CallDiagnosticMetrics, BehavioralMetrics, OperationalMetrics, BusinessMetrics, RtcMetrics, };
-export type { ClientEvent, ClientEventLeaveReason, SubmitBehavioralEvent, SubmitClientEvent, SubmitInternalEvent, SubmitMQE, SubmitOperationalEvent, PreComputedLatencies, };
+export type { ClientEvent, ClientEventLeaveReason, SubmitBehavioralEvent, SubmitClientEvent, SubmitInternalEvent, SubmitMQE, SubmitOperationalEvent, SubmitBusinessEvent, PreComputedLatencies, };

package/dist/types/metrics.types.d.ts

@@ -6,11 +6,14 @@ export type ClientEventError = NonNullable<RawClientEvent['errors']>[0];
 export type EnvironmentType = NonNullable<RawEvent['origin']['environment']>;
 export type NewEnvironmentType = NonNullable<RawEvent['origin']['newEnvironment']>;
 export type ClientLaunchMethodType = NonNullable<RawEvent['origin']['clientInfo']>['clientLaunchMethod'];
+export type ClientUserNameInput = NonNullable<RawClientEvent['userNameInput']>;
+export type ClientEmailInput = NonNullable<RawClientEvent['emailInput']>;
 export type BrowserLaunchMethodType = NonNullable<RawEvent['origin']['clientInfo']>['browserLaunchMethod'];
-export type MetricEventProduct = 'webex' | 'wxcc_desktop';
+export type MetricEventProduct = 'webex' | 'wxcc_desktop' | 'wxcc_crm' | 'wxcc_sdk';
 export type MetricEventAgent = 'user' | 'browser' | 'system' | 'sdk' | 'redux' | 'service' | 'api';
 export type MetricEventVerb = 'abort' | 'accept' | 'activate' | 'apply' | 'answer' | 'authorize' | 'build' | 'cancel' | 'change' | 'click' | 'close' | 'complete' | 'connect' | 'create' | 'deactivate' | 'decrypt' | 'delete' | 'deliver' | 'destroy' | 'disable' | 'disconnect' | 'dismiss' | 'display' | 'download' | 'edit' | 'enable' | 'encrypt' | 'end' | 'expire' | 'fail' | 'fetch' | 'fire' | 'generate' | 'get' | 'hide' | 'hover' | 'ignore' | 'initialize' | 'initiate' | 'invalidate' | 'join' | 'list' | 'load' | 'login' | 'logout' | 'notify' | 'offer' | 'open' | 'press' | 'receive' | 'refer' | 'refresh' | 'register' | 'release' | 'reload' | 'reject' | 'request' | 'reset' | 'resize' | 'respond' | 'retry' | 'revoke' | 'save' | 'search' | 'select' | 'send' | 'set' | 'sign' | 'start' | 'submit' | 'switch' | 'sync' | 'toggle' | 'transfer' | 'unregister' | 'update' | 'upload' | 'use' | 'validate' | 'view' | 'visit' | 'wait' | 'warn' | 'exit';
 export type MetricEventJoinFlowVersion = 'Other' | 'NewFTE';
+export type MetricEventMeetingJoinPhase = 'pre-join' | 'join' | 'in-meeting';
 export type SubmitClientEventOptions = {
     meetingId?: string;
     mediaConnections?: any[];
@@ -25,6 +28,10 @@ export type SubmitClientEventOptions = {
     webexConferenceIdStr?: string;
     globalMeetingId?: string;
     joinFlowVersion?: MetricEventJoinFlowVersion;
+    meetingJoinPhase?: MetricEventMeetingJoinPhase;
+    triggeredTime?: string;
+    emailInput?: ClientEmailInput;
+    userNameInput?: ClientUserNameInput;
 };
 export type SubmitMQEOptions = {
     meetingId: string;
@@ -129,6 +136,10 @@ export type SubmitOperationalEvent = (args: {
     name: OperationalEvent['metricName'];
     payload: EventPayload;
 }) => void;
+export type SubmitBusinessEvent = (args: {
+    name: OperationalEvent['metricName'];
+    payload: EventPayload;
+}) => void;
 export type SubmitMQE = (args: {
     name: MediaQualityEvent['name'];
     payload: SubmitMQEPayload;
@@ -154,4 +165,9 @@ export interface IMetricsAttributes {
     meetingId?: string;
     callId?: string;
 }
+export interface DelayedClientEvent {
+    name: ClientEvent['name'];
+    payload?: RecursivePartial<ClientEvent['payload']>;
+    options?: SubmitClientEventOptions;
+}
 export {};

package/dist/types/new-metrics.d.ts

@@ -18,6 +18,10 @@ declare class Metrics extends WebexPlugin {
     operationalMetrics: OperationalMetrics;
     businessMetrics: BusinessMetrics;
     isReady: boolean;
+    /**
+     * Whether or not to delay the submission of client events.
+     */
+    delaySubmitClientEvents: boolean;
     /**
      * Constructor
      * @param args
@@ -60,7 +64,7 @@ declare class Metrics extends WebexPlugin {
      */
     isReadyToSubmitOperationalEvents(): boolean;
     /**
-     * @returns true once we have the deviceId we need to submit buisness events
+     * @returns true once we have the deviceId we need to submit business events
      */
     isReadyToSubmitBusinessEvents(): boolean;
     /**
@@ -83,13 +87,14 @@ declare class Metrics extends WebexPlugin {
         payload?: EventPayload;
     }): void | Promise<void>;
     /**
-     * Buisness event
+     * Business event
      * @param args
      */
-    submitBusinessEvent({ name, payload, table, }: {
+    submitBusinessEvent({ name, payload, table, metadata, }: {
         name: string;
         payload: EventPayload;
         table?: Table;
+        metadata?: EventPayload;
     }): Promise<void>;
     /**
      * Call Analyzer: Media Quality Event
@@ -169,5 +174,12 @@ declare class Metrics extends WebexPlugin {
      * @returns {boolean}
      */
     isServiceErrorExpected(serviceErrorCode: number): boolean;
+    /**
+     * Sets the value of delaySubmitClientEvents. If set to true, client events will be delayed until submitDelayedClientEvents is called. If
+     * set to false, delayed client events will be submitted.
+     *
+     * @param {boolean} shouldDelay - A boolean value indicating whether to delay the submission of client events.
+     */
+    setDelaySubmitClientEvents(shouldDelay: boolean): Promise<any[]> | Promise<void>;
 }
 export default Metrics;

package/package.json

@@ -26,22 +26,22 @@
     "@webex/eslint-config-legacy": "0.0.0",
     "@webex/jest-config-legacy": "0.0.0",
     "@webex/legacy-tools": "0.0.0",
-    "@webex/test-helper-chai": "3.7.0",
-    "@webex/test-helper-mocha": "3.7.0",
-    "@webex/test-helper-mock-webex": "3.7.0",
-    "@webex/test-helper-test-users": "3.7.0",
+    "@webex/test-helper-chai": "3.8.0-next.10",
+    "@webex/test-helper-mocha": "3.8.0-next.10",
+    "@webex/test-helper-mock-webex": "3.8.0-next.10",
+    "@webex/test-helper-test-users": "3.8.0-next.10",
     "eslint": "^8.24.0",
     "prettier": "^2.7.1",
     "sinon": "^9.2.4"
   },
   "dependencies": {
-    "@webex/common": "3.7.0",
-    "@webex/common-timers": "3.7.0",
-    "@webex/event-dictionary-ts": "^1.0.1594",
-    "@webex/internal-plugin-metrics": "3.7.0",
-    "@webex/test-helper-chai": "3.7.0",
-    "@webex/test-helper-mock-webex": "3.7.0",
-    "@webex/webex-core": "3.7.0",
+    "@webex/common": "3.8.0-next.10",
+    "@webex/common-timers": "3.8.0-next.10",
+    "@webex/event-dictionary-ts": "^1.0.1688",
+    "@webex/internal-plugin-metrics": "3.8.0-next.10",
+    "@webex/test-helper-chai": "3.8.0-next.10",
+    "@webex/test-helper-mock-webex": "3.8.0-next.10",
+    "@webex/webex-core": "3.8.0-next.10",
     "ip-anonymize": "^0.1.0",
     "lodash": "^4.17.21",
     "uuid": "^3.3.2"
@@ -54,5 +54,5 @@
     "test:style": "eslint ./src/**/*.*",
     "test:unit": "webex-legacy-tools test --unit --runner mocha"
   },
-  "version": "3.7.0"
+  "version": "3.8.0-next.10"
 }

package/src/business-metrics.ts

@@ -2,121 +2,111 @@ import GenericMetrics from './generic-metrics';
 import {EventPayload, Table} from './metrics.types';
 
 /**
- * @description Util class to handle Buisness Metrics
+ * @description Util class to handle Business Metrics
  * @export
  * @class BusinessMetrics
  */
 export default class BusinessMetrics extends GenericMetrics {
   /**
-   * unfortunately, the pinot team does not allow changes to the schema of wbxapp_callend_metrics
-   * so we have to shim this layer specifically for this
-   * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table+wbxapp_callend_metrics
-   * @param {EventPayload} payload payload of the metric
-   * @returns {Promise<any>}
-   */
-  private submitCallEndEvent({payload}: {payload: EventPayload}) {
-    const event = {
-      type: ['business'],
-      eventPayload: {
-        key: 'callEnd',
-        client_timestamp: new Date().toISOString(),
-        appType: 'Web Client',
-        value: {
-          ...payload,
-        },
-      },
-    };
-
-    return this.submitEvent({
-      kind: 'buisness-events:wbxapp_callend_metrics -> ',
-      name: 'wbxapp_callend_metrics',
-      event,
-    });
-  }
-
-  /**
-   * Submit a buisness metric to our metrics endpoint, going to the default business_ucf table
-   * all event payload keys are converted into a hex string value
-   * unfortunately, the pinot team does not allow changes to the schema of business_metrics
-   * so we have to shim this layer specifically for this
-   * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table%3A+business_metrics
+   * Build the metric event to submit.
    * @param {string} name of the metric
-   * @param {EventPayload} payload payload of the metric
-   * @returns {Promise<any>}
-   */
-  private submitBusinessMetricsEvent({name, payload}: {name: string; payload: EventPayload}) {
-    const event = {
-      type: ['business'],
-      eventPayload: {
-        key: name,
-        client_timestamp: new Date().toISOString(),
-        appType: 'Web Client',
-        value: {
-          ...this.getContext(),
-          ...this.getBrowserDetails(),
-          ...payload,
-        },
-      },
-    };
-
-    return this.submitEvent({kind: 'buisness-events:business_metrics -> ', name, event});
-  }
-
-  /**
-   * Submit a buisness metric to our metrics endpoint, going to the default business_ucf table
-   * all event payload keys are converted into a hex string value
-   * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Business+metrics++-%3E+ROMA
-   * @param {string} name of the metric
-   * @param {EventPayload} user payload of the metric
-   * @returns {Promise<any>}
+   * @param {EventPayload} payload user payload of the metric
+   * @param {EventPayload} metadata to include outside of eventPayload.value
+   * @returns {MetricEvent} The constructed metric event
    */
-  private submitDefaultEvent({name, payload}: {name: string; payload: EventPayload}) {
-    const event = {
+  private buildEvent({name, payload, metadata}: {name: string; payload: object; metadata: object}) {
+    return {
       type: ['business'],
       eventPayload: {
         key: name,
-        appType: 'Web Client',
         client_timestamp: new Date().toISOString(),
-        context: this.getContext(),
-        browserDetails: this.getBrowserDetails(),
+        ...metadata,
         value: payload,
       },
     };
-
-    return this.submitEvent({kind: 'buisness-events:default -> ', name, event});
   }
 
   /**
-   * Submit a buisness metric to our metrics endpoint.
+   * Submit a business metric to our metrics endpoint.
    * routes to the correct table with the correct schema payload by table
    * https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Business+metrics++-%3E+ROMA
    * @param {string} name of the metric, ignored if going to wbxapp_callend_metrics
    * @param {EventPayload} payload user payload of the metric
    * @param {Table} table optional - to submit the metric to and adapt the sent schema
+   * @param {EventPayload} metadata optional - to include outside of eventPayload.value
    * @returns {Promise<any>}
    */
   public submitBusinessEvent({
     name,
     payload,
     table,
+    metadata,
   }: {
     name: string;
     payload: EventPayload;
     table?: Table;
+    metadata?: EventPayload;
   }): Promise<void> {
     if (!table) {
       table = 'default';
     }
+    if (!metadata) {
+      metadata = {};
+    }
+    if (!metadata.appType) {
+      metadata.appType = 'Web Client';
+    }
     switch (table) {
-      case 'wbxapp_callend_metrics':
-        return this.submitCallEndEvent({payload});
-      case 'business_metrics':
-        return this.submitBusinessMetricsEvent({name, payload});
+      case 'wbxapp_callend_metrics': {
+        // https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table+wbxapp_callend_metrics
+        const callEndEvent = this.buildEvent({name: 'callEnd', payload, metadata});
+
+        return this.submitEvent({
+          kind: 'business-events:wbxapp_callend_metrics -> ',
+          name: 'wbxapp_callend_metrics',
+          event: callEndEvent,
+        });
+      }
+
+      case 'business_metrics': {
+        // all event payload keys are converted into a hex string value
+        // unfortunately, the pinot team does not allow changes to the schema of business_metrics
+        // so we have to shim this layer specifically for this
+        // https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Table%3A+business_metrics
+        const businessEvent = this.buildEvent({
+          name,
+          payload: {
+            ...this.getContext(),
+            ...this.getBrowserDetails(),
+            ...payload,
+          },
+          metadata,
+        });
+
+        return this.submitEvent({
+          kind: 'business-events:business_metrics -> ',
+          name,
+          event: businessEvent,
+        });
+      }
+
       case 'business_ucf':
-        return this.submitDefaultEvent({name, payload});
       case 'default':
-      default:
-        return this.submitDefaultEvent({name, payload});
+      default: {
+        // all event payload keys are converted into a hex string value
+        // https://confluence-eng-gpk2.cisco.com/conf/display/WAP/Business+metrics++-%3E+ROMA
+        const defaultEvent = this.buildEvent({
+          name,
+          payload,
+          metadata: {
+            context: this.getContext(),
+            browserDetails: this.getBrowserDetails(),
+            ...metadata,
+          },
+        });
+
+        return this.submitEvent({kind: 'business-events:default -> ', name, event: defaultEvent});
+      }
     }
   }
 }

package/src/call-diagnostic/call-diagnostic-metrics-latencies.ts

@@ -83,11 +83,16 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
       key === 'client.media.rx.start' ||
       key === 'client.media.tx.start' ||
       key === 'internal.client.meetinginfo.request' ||
-      key === 'internal.client.meetinginfo.response'
+      key === 'internal.client.meetinginfo.response' ||
+      key === 'client.media-engine.remote-sdp-received'
     ) {
       this.saveFirstTimestampOnly(key, value);
     } else {
       this.latencyTimestamps.set(key, value);
+      // new offer/answer so reset the remote SDP timestamp
+      if (key === 'client.media-engine.local-sdp-generated') {
+        this.latencyTimestamps.delete('client.media-engine.remote-sdp-received');
+      }
     }
   }
 
@@ -146,7 +151,8 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
   public getDiffBetweenTimestamps(a: MetricEventNames, b: MetricEventNames) {
     const start = this.latencyTimestamps.get(a);
     const end = this.latencyTimestamps.get(b);
-    if (start && end) {
+
+    if (typeof start === 'number' && typeof end === 'number') {
       return end - start;
     }
 
@@ -188,7 +194,7 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
   public getU2CTime() {
     const u2cLatency = this.precomputedLatencies.get('internal.get.u2c.time');
 
-    return u2cLatency ? Math.floor(u2cLatency) : undefined;
+    return typeof u2cLatency === 'number' ? Math.floor(u2cLatency) : undefined;
   }
 
   /**
@@ -291,7 +297,9 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
    * @returns - latency
    */
   public getPageJMT() {
-    return this.precomputedLatencies.get('internal.client.pageJMT') || undefined;
+    const latency = this.precomputedLatencies.get('internal.client.pageJMT');
+
+    return typeof latency === 'number' ? latency : undefined;
   }
 
   /**
@@ -299,7 +307,9 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
    * @returns - latency
    */
   public getDownloadTimeJMT() {
-    return this.precomputedLatencies.get('internal.download.time') || undefined;
+    const latency = this.precomputedLatencies.get('internal.download.time');
+
+    return typeof latency === 'number' ? latency : undefined;
   }
 
   /**
@@ -315,8 +325,15 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
       );
     }
 
-    // for cross launch and guest flows
-    return this.precomputedLatencies.get('internal.click.to.interstitial') || undefined;
+    const clickToInterstitialLatency = this.precomputedLatencies.get(
+      'internal.click.to.interstitial'
+    );
+
+    if (typeof clickToInterstitialLatency === 'number') {
+      return clickToInterstitialLatency;
+    }
+
+    return undefined;
   }
 
   /**
@@ -353,7 +370,8 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
     // get the first timestamp
     const connectedMedia = this.latencyTimestamps.get('client.ice.end');
 
-    const lobbyTime = this.getStayLobbyTime() || 0;
+    const lobbyTimeLatency = this.getStayLobbyTime();
+    const lobbyTime = typeof lobbyTimeLatency === 'number' ? lobbyTimeLatency : 0;
 
     if (interstitialJoinClickTimestamp && connectedMedia) {
       return connectedMedia - interstitialJoinClickTimestamp - lobbyTime;
@@ -370,7 +388,7 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
     const clickToInterstitial = this.getClickToInterstitial();
     const interstitialToJoinOk = this.getInterstitialToJoinOK();
 
-    if (clickToInterstitial && interstitialToJoinOk) {
+    if (typeof clickToInterstitial === 'number' && typeof interstitialToJoinOk === 'number') {
       return clickToInterstitial + interstitialToJoinOk;
     }
 
@@ -422,7 +440,7 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
     const interstitialToJoinOk = this.getInterstitialToJoinOK();
     const joinConfJMT = this.getJoinConfJMT();
 
-    if (interstitialToJoinOk && joinConfJMT) {
+    if (typeof interstitialToJoinOk === 'number' && typeof joinConfJMT === 'number') {
       return interstitialToJoinOk - joinConfJMT;
     }
 
@@ -449,7 +467,9 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
   public getReachabilityClustersReqResp() {
     const reachablityClusterReqResp = this.precomputedLatencies.get('internal.get.cluster.time');
 
-    return reachablityClusterReqResp ? Math.floor(reachablityClusterReqResp) : undefined;
+    return typeof reachablityClusterReqResp === 'number'
+      ? Math.floor(reachablityClusterReqResp)
+      : undefined;
   }
 
   /**
@@ -472,7 +492,7 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
   public getExchangeCITokenJMT() {
     const exchangeCITokenJMT = this.precomputedLatencies.get('internal.exchange.ci.token.time');
 
-    return exchangeCITokenJMT ? Math.floor(exchangeCITokenJMT) : undefined;
+    return typeof exchangeCITokenJMT === 'number' ? Math.floor(exchangeCITokenJMT) : undefined;
   }
 
   /**
@@ -481,7 +501,9 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
   public getRefreshCaptchaReqResp() {
     const refreshCaptchaReqResp = this.precomputedLatencies.get('internal.refresh.captcha.time');
 
-    return refreshCaptchaReqResp ? Math.floor(refreshCaptchaReqResp) : undefined;
+    return typeof refreshCaptchaReqResp === 'number'
+      ? Math.floor(refreshCaptchaReqResp)
+      : undefined;
   }
 
   /**
@@ -493,7 +515,7 @@ export default class CallDiagnosticLatencies extends WebexPlugin {
       'internal.api.fetch.intelligence.models'
     );
 
-    return downloadIntelligenceModelsReqResp
+    return typeof downloadIntelligenceModelsReqResp === 'number'
       ? Math.floor(downloadIntelligenceModelsReqResp)
       : undefined;
   }