@webex/plugin-meetings 1.159.3 → 1.159.6

package/src/meetings/index.js

@@ -1,6 +1,3 @@
-/*!
-  * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
-*/
 
 import '@webex/internal-plugin-mercury';
 import '@webex/internal-plugin-conversation';
@@ -109,43 +106,43 @@ import MeetingsUtil from './util';
    * @class
    */
 export default class Meetings extends WebexPlugin {
-    namespace = MEETINGS;
+  namespace = MEETINGS;
 
-    /**
+  /**
      * Initializes the Meetings Plugin
      * @constructor
      * @public
      * @memberof Meetings
      */
-    constructor(...args) {
-      super(...args);
+  constructor(...args) {
+    super(...args);
 
-      /**
+    /**
        * The Meetings request to interact with server
        * @instance
        * @type {Object}
        * @private
        * @memberof Meetings
        */
-      this.request = new Request({}, {parent: this.webex});
-      /**
+    this.request = new Request({}, {parent: this.webex});
+    /**
        * Log upload request helper
        * @instance
        * @type {Object}
        * @private
        * @memberof Meetings
        */
-      this.loggerRequest = new LoggerRequest({webex: this.webex});
-      this.meetingCollection = new MeetingCollection();
-      /**
+    this.loggerRequest = new LoggerRequest({webex: this.webex});
+    this.meetingCollection = new MeetingCollection();
+    /**
        * The PersonalMeetingRoom object to interact with server
        * @instance
        * @type {Object}
        * @public
        * @memberof Meetings
        */
-      this.personalMeetingRoom = null;
-      /**
+    this.personalMeetingRoom = null;
+    /**
        * The Reachability object to interact with server, starts as null until {@link Meeting#setReachability} is called
        * starts as null
        * @instance
@@ -153,27 +150,27 @@ export default class Meetings extends WebexPlugin {
        * @private
        * @memberof Meetings
        */
-      this.reachability = null;
+    this.reachability = null;
 
-      /**
+    /**
        * If the meetings plugin has been registered and listening via {@link Meetings#register}
        * @instance
        * @type {Boolean}
        * @public
        * @memberof Meetings
        */
-      this.registered = false;
+    this.registered = false;
 
-      /**
+    /**
        * This values indicates the preferred webex site the user will start there meeting, getsits value from {@link Meetings#register}
        * @instance
        * @type {String}
        * @private
        * @memberof Meetings
        */
-      this.preferredWebexSite = '';
+    this.preferredWebexSite = '';
 
-      /**
+    /**
        * The public interface for the internal Media util files. These are helpful to expose outside the context
        * of a meeting so that a user can access media without creating a meeting instance.
        * @instance
@@ -181,17 +178,17 @@ export default class Meetings extends WebexPlugin {
        * @private
        * @memberof Meetings
        */
-      this.media = {
-        getUserMedia: Media.getUserMedia,
-        getSupportedDevice: Media.getSupportedDevice
-      };
-
-      this.onReady();
-      MeetingsUtil.checkH264Support({disableNotifications: true});
-      Metrics.initialSetup(this.meetingCollection, this.webex);
-    }
+    this.media = {
+      getUserMedia: Media.getUserMedia,
+      getSupportedDevice: Media.getSupportedDevice
+    };
 
-    /**
+    this.onReady();
+    MeetingsUtil.checkH264Support({disableNotifications: true});
+    Metrics.initialSetup(this.meetingCollection, this.webex);
+  }
+
+  /**
      * handle locus events and takes meeting actions with them as they come in
      * @param {Object} data a locus event
      * @param {String} data.locusUrl
@@ -201,102 +198,102 @@ export default class Meetings extends WebexPlugin {
      * @private
      * @memberof Meetings
      */
-    handleLocusEvent(data) {
-      let meeting = null;
+  handleLocusEvent(data) {
+    let meeting = null;
 
-      // getting meeting by correlationId. This will happen for the new event
-      // Either the locus
-      // TODO : Add check for the callBack Address
-      meeting = this.meetingCollection.getByKey(LOCUS_URL, data.locusUrl) ||
+    // getting meeting by correlationId. This will happen for the new event
+    // Either the locus
+    // TODO : Add check for the callBack Address
+    meeting = this.meetingCollection.getByKey(LOCUS_URL, data.locusUrl) ||
       this.meetingCollection.getByKey(CORRELATION_ID, MeetingsUtil.checkForCorrelationId(this.webex.internal.device.url, data.locus)) ||
       this.meetingCollection.getByKey(SIP_URI, data.locus.self && data.locus.self.callbackInfo && data.locus.self.callbackInfo.callbackAddress) ||
-      (this.config.experimental.enableUnifiedMeetings ? undefined : this.meetingCollection.getByKey(CONVERSATION_URL, data.locus.conversationUrl));
+      (data.locus.info?.isUnifiedSpaceMeeting ? undefined : this.meetingCollection.getByKey(CONVERSATION_URL, data.locus.conversationUrl));
 
-      // Special case when locus has got replaced, This only happend once if a replace locus exists
-      // https://sqbu-github.cisco.com/WebExSquared/locus/wiki/Locus-changing-mid-call
+    // Special case when locus has got replaced, This only happend once if a replace locus exists
+    // https://sqbu-github.cisco.com/WebExSquared/locus/wiki/Locus-changing-mid-call
 
-      if (!meeting && data.locus?.replaces?.length > 0) {
-        // Always the last element in the replace is the active one
-        meeting = this.meetingCollection.getByKey(LOCUS_URL, data.locus.replaces[data.locus.replaces.length - 1].locusUrl);
-      }
+    if (!meeting && data.locus?.replaces?.length > 0) {
+      // Always the last element in the replace is the active one
+      meeting = this.meetingCollection.getByKey(LOCUS_URL, data.locus.replaces[data.locus.replaces.length - 1].locusUrl);
+    }
 
-      if (!meeting) {
-        // TODO: create meeting when we get a meeting object
-        // const checkForEnded = (locus) => {
-        // TODO: you already ended the meeting but you got an event later
-        // Mainly for 1:1 Callsor meeting
-        // Happens mainly after refresh
+    if (!meeting) {
+      // TODO: create meeting when we get a meeting object
+      // const checkForEnded = (locus) => {
+      // TODO: you already ended the meeting but you got an event later
+      // Mainly for 1:1 Callsor meeting
+      // Happens mainly after refresh
 
-        // 1:1 Meeting
-        // 1)  You ended a call before but you got a mercury event
-        // Make sure end the call and cleanup the meeting only if the mercury
-        // event says so
-        // 2) Maintain lastSync time in the meetings object which helps to compare
-        // If the meeting came befor or after the sync . ANy meeting start time before the sync time is invalid
+      // 1:1 Meeting
+      // 1)  You ended a call before but you got a mercury event
+      // Make sure end the call and cleanup the meeting only if the mercury
+      // event says so
+      // 2) Maintain lastSync time in the meetings object which helps to compare
+      // If the meeting came befor or after the sync . ANy meeting start time before the sync time is invalid
 
-        // For space Meeting
-        // Check the locus object and see who has joined
+      // For space Meeting
+      // Check the locus object and see who has joined
 
-        // };
-        // rather then locus object change to locus url
+      // };
+      // rather then locus object change to locus url
 
-        if (data.locus && data.locus.fullState && data.locus.fullState.state === LOCUS.STATE.INACTIVE) {
-          // just ignore the event as its already ended and not active
-          LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Locus event received for meeting, after it was ended.');
+      if (data.locus && data.locus.fullState && data.locus.fullState.state === LOCUS.STATE.INACTIVE) {
+        // just ignore the event as its already ended and not active
+        LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Locus event received for meeting, after it was ended.');
 
-          return;
-        }
+        return;
+      }
 
 
-        // When its wireless share or guest and user leaves the meeting we dont have to keep the meeting object
-        // Any future events will be neglected
+      // When its wireless share or guest and user leaves the meeting we dont have to keep the meeting object
+      // Any future events will be neglected
 
-        if (data.locus && data.locus.self && (data.locus.self.state === _LEFT_ && data.locus.self.removed === true)) {
-          // just ignore the event as its already ended and not active
-          LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Locus event received for meeting, after it was ended.');
+      if (data.locus && data.locus.self && (data.locus.self.state === _LEFT_ && data.locus.self.removed === true)) {
+        // just ignore the event as its already ended and not active
+        LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Locus event received for meeting, after it was ended.');
 
-          return;
-        }
-
-        this.create(data.locus, _LOCUS_ID_).then((newMeeting) => {
-          meeting = newMeeting;
-
-          // It's a new meeting so initialize the locus data
-          meeting.locusInfo.initialSetup(data.locus);
-        }).catch((e) => {
-          console.log(e);
-        })
-          .finally(() => {
-            // There will be cases where locus event comes in gets created and deleted because its a 1:1 and meeting gets deleted
-            // because the other user left so before sending 'added' event make sure it exists in the collection
-
-            if (this.getMeetingByType(_ID_, meeting.id)) {
-              Metrics.postEvent({event: eventType.REMOTE_STARTED, meeting, data: {trigger: trigger.MERCURY_EVENT}});
-              Trigger.trigger(
-                this,
-                {
-                  file: 'meetings',
-                  function: 'handleLocusEvent'
-                },
-                EVENT_TRIGGERS.MEETING_ADDED,
-                {
-                  meeting,
-                  type: meeting.type === _MEETING_ ? _JOIN_ : _INCOMING_
-                }
-              );
-            }
-            else {
-              // Meeting got added but was not found in the collection. It might have got destroyed
-              LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Created and destroyed meeting object before sending an event');
-            }
-          });
-      }
-      else {
-        meeting.locusInfo.parse(meeting, data);
+        return;
       }
+
+      this.create(data.locus, _LOCUS_ID_).then((newMeeting) => {
+        meeting = newMeeting;
+
+        // It's a new meeting so initialize the locus data
+        meeting.locusInfo.initialSetup(data.locus);
+      }).catch((e) => {
+        console.log(e);
+      })
+        .finally(() => {
+          // There will be cases where locus event comes in gets created and deleted because its a 1:1 and meeting gets deleted
+          // because the other user left so before sending 'added' event make sure it exists in the collection
+
+          if (this.getMeetingByType(_ID_, meeting.id)) {
+            Metrics.postEvent({event: eventType.REMOTE_STARTED, meeting, data: {trigger: trigger.MERCURY_EVENT}});
+            Trigger.trigger(
+              this,
+              {
+                file: 'meetings',
+                function: 'handleLocusEvent'
+              },
+              EVENT_TRIGGERS.MEETING_ADDED,
+              {
+                meeting,
+                type: meeting.type === _MEETING_ ? _JOIN_ : _INCOMING_
+              }
+            );
+          }
+          else {
+            // Meeting got added but was not found in the collection. It might have got destroyed
+            LoggerProxy.logger.warn('Meetings:index#handleLocusEvent --> Created and destroyed meeting object before sending an event');
+          }
+        });
     }
+    else {
+      meeting.locusInfo.parse(meeting, data);
+    }
+  }
 
-    /**
+  /**
      * handles locus events through mercury that are not roap
      * @param {Object} envelope
      * @param {Object} envelope.data
@@ -305,135 +302,135 @@ export default class Meetings extends WebexPlugin {
      * @private
      * @memberof Meetings
      */
-    handleLocusMercury(envelope) {
-      const {data} = envelope;
-      const {eventType} = data;
+  handleLocusMercury(envelope) {
+    const {data} = envelope;
+    const {eventType} = data;
 
-      if (eventType && eventType !== LOCUSEVENT.MESSAGE_ROAP) {
-        this.handleLocusEvent(data);
-      }
+    if (eventType && eventType !== LOCUSEVENT.MESSAGE_ROAP) {
+      this.handleLocusEvent(data);
     }
+  }
 
 
-    /**
+  /**
      * handles mecury offline event
      * @returns {undefined}
      * @private
      * @memberof Meetings
      */
-    handleMercuryOffline() {
-      Trigger.trigger(
-        this,
-        {
-          file: 'meetings/index',
-          function: 'handleMercuryOffline'
-        },
-        EVENT_TRIGGERS.MEETINGS_NETWORK_DISCONNECTED,
-      );
-    }
-
-
-    /**
+  handleMercuryOffline() {
+    Trigger.trigger(
+      this,
+      {
+        file: 'meetings/index',
+        function: 'handleMercuryOffline'
+      },
+      EVENT_TRIGGERS.MEETINGS_NETWORK_DISCONNECTED,
+    );
+  }
+
+
+  /**
      * registers for locus and roap mercury events
      * @returns {undefined}
      * @private
      * @memberof Meetings
      */
-    listenForEvents() {
-      this.webex.internal.mercury.on(LOCUSEVENT.LOCUS_MERCURY, (envelope) => {
-        this.handleLocusMercury(envelope);
-      });
-      this.webex.internal.mercury.on(ROAP.ROAP_MERCURY, (envelope) => {
-        MeetingsUtil.handleRoapMercury(envelope, this.meetingCollection);
-      });
-
-      this.webex.internal.mercury.on(ONLINE, () => {
-        this.syncMeetings();
-      });
-
-      this.webex.internal.mercury.on(OFFLINE, () => {
-        this.handleMercuryOffline();
-      });
-    }
-
-    /**
+  listenForEvents() {
+    this.webex.internal.mercury.on(LOCUSEVENT.LOCUS_MERCURY, (envelope) => {
+      this.handleLocusMercury(envelope);
+    });
+    this.webex.internal.mercury.on(ROAP.ROAP_MERCURY, (envelope) => {
+      MeetingsUtil.handleRoapMercury(envelope, this.meetingCollection);
+    });
+
+    this.webex.internal.mercury.on(ONLINE, () => {
+      this.syncMeetings();
+    });
+
+    this.webex.internal.mercury.on(OFFLINE, () => {
+      this.handleMercuryOffline();
+    });
+  }
+
+  /**
      * stops listening for locus and roap mercury events
      * @returns {undefined}
      * @private
      * @memberof Meetings
      */
-    stopListeningForEvents() {
-      this.webex.internal.mercury.off(LOCUSEVENT.LOCUS_MERCURY);
-      this.webex.internal.mercury.off(ROAP.ROAP_MERCURY);
-      this.webex.internal.mercury.off(ONLINE);
-    }
+  stopListeningForEvents() {
+    this.webex.internal.mercury.off(LOCUSEVENT.LOCUS_MERCURY);
+    this.webex.internal.mercury.off(ROAP.ROAP_MERCURY);
+    this.webex.internal.mercury.off(ONLINE);
+  }
 
-    /**
+  /**
      * @returns {undefined}
      * @private
      * @memberof Meetings
      */
-    onReady() {
-      this.webex.once(READY, () => {
-        StaticConfig.set(this.config);
-        LoggerProxy.set(this.webex.logger);
-        LoggerConfig.set(this.config.logging);
+  onReady() {
+    this.webex.once(READY, () => {
+      StaticConfig.set(this.config);
+      LoggerProxy.set(this.webex.logger);
+      LoggerConfig.set(this.config.logging);
 
-        /**
+      /**
        * The MeetingInfo object to interact with server
        * @instance
        * @type {Object}
        * @private
        * @memberof Meetings
        */
-        this.meetingInfo = this.config.experimental.enableUnifiedMeetings ? new MeetingInfoV2(this.webex) : new MeetingInfo(this.webex);
-        this.personalMeetingRoom = new PersonalMeetingRoom({meetingInfo: this.meetingInfo}, {parent: this.webex});
+      this.meetingInfo = this.config.experimental.enableUnifiedMeetings ? new MeetingInfoV2(this.webex) : new MeetingInfo(this.webex);
+      this.personalMeetingRoom = new PersonalMeetingRoom({meetingInfo: this.meetingInfo}, {parent: this.webex});
 
-        Trigger.trigger(
-          this,
-          {
-            file: 'meetings',
-            function: 'onReady'
-          },
-          EVENT_TRIGGERS.MEETINGS_READY
-        );
-      });
-    }
+      Trigger.trigger(
+        this,
+        {
+          file: 'meetings',
+          function: 'onReady'
+        },
+        EVENT_TRIGGERS.MEETINGS_READY
+      );
+    });
+  }
 
-    /**
+  /**
      * API to toggle unified meetings
      * @param {Boolean} changeState
      * @private
      * @memberof Meetings
      * @returns {undefined}
     */
-    _toggleUnifiedMeetings(changeState) {
-      if (typeof changeState !== 'boolean') {
-        return;
-      }
-      if (this.config?.experimental?.enableUnifiedMeetings !== changeState) {
-        this.config.experimental.enableUnifiedMeetings = changeState;
-        this.meetingInfo = changeState ? new MeetingInfoV2(this.webex) : new MeetingInfo(this.webex);
-      }
+  _toggleUnifiedMeetings(changeState) {
+    if (typeof changeState !== 'boolean') {
+      return;
     }
+    if (this.config?.experimental?.enableUnifiedMeetings !== changeState) {
+      this.config.experimental.enableUnifiedMeetings = changeState;
+      this.meetingInfo = changeState ? new MeetingInfoV2(this.webex) : new MeetingInfo(this.webex);
+    }
+  }
 
-    /**
+  /**
      * API to toggle starting adhoc meeting
      * @param {Boolean} changeState
      * @private
      * @memberof Meetings
      * @returns {undefined}
     */
-    _toggleAdhocMeetings(changeState) {
-      if (typeof changeState !== 'boolean') {
-        return;
-      }
-      if (this.config?.experimental?.enableAdhocMeetings !== changeState) {
-        this.config.experimental.enableAdhocMeetings = changeState;
-      }
+  _toggleAdhocMeetings(changeState) {
+    if (typeof changeState !== 'boolean') {
+      return;
+    }
+    if (this.config?.experimental?.enableAdhocMeetings !== changeState) {
+      this.config.experimental.enableAdhocMeetings = changeState;
     }
+  }
 
-    /**
+  /**
      * Explicitly sets up the meetings plugin by registering
      * the device, connecting to mercury, and listening for locus events.
      *
@@ -441,61 +438,61 @@ export default class Meetings extends WebexPlugin {
      * @public
      * @memberof Meetings
      */
-    register() {
-      if (!this.webex.canAuthorize) {
-        LoggerProxy.logger.error('Meetings:index#register --> ERROR, Unable to register, SDK cannot authorize');
+  register() {
+    if (!this.webex.canAuthorize) {
+      LoggerProxy.logger.error('Meetings:index#register --> ERROR, Unable to register, SDK cannot authorize');
 
-        return Promise.reject(new Error('SDK cannot authorize'));
-      }
+      return Promise.reject(new Error('SDK cannot authorize'));
+    }
 
 
-      if (this.registered) {
-        LoggerProxy.logger.info('Meetings:index#register --> INFO, Meetings plugin already registered');
+    if (this.registered) {
+      LoggerProxy.logger.info('Meetings:index#register --> INFO, Meetings plugin already registered');
 
-        return Promise.resolve();
-      }
+      return Promise.resolve();
+    }
+
+    return Promise.all([
+      this.fetchUserPreferredWebexSite(),
+      this.getGeoHint(),
+      this.startReachability().catch((error) => {
+        LoggerProxy.logger.error(`Meetings:index#register --> GDM error, ${error.message}`);
+      }),
+      this.webex.internal.device.register()
+        .then(() => LoggerProxy.logger.info(`Meetings:index#register --> INFO, Device registered ${this.webex.internal.device.url}`))
+        .then(() => this.webex.internal.mercury.connect()),
+      MeetingsUtil.checkH264Support.call(this)
+    ]).then(() => {
+      this.listenForEvents();
+      Trigger.trigger(
+        this,
+        {
+          file: 'meetings',
+          function: 'register'
+        },
+        EVENT_TRIGGERS.MEETINGS_REGISTERED
+      );
+      this.registered = true;
+      Metrics.sendBehavioralMetric(
+        BEHAVIORAL_METRICS.MEETINGS_REGISTRATION_SUCCESS,
+      );
+    })
+      .catch((error) => {
+        LoggerProxy.logger.error(`Meetings:index#register --> ERROR, Unable to register, ${error.message}`);
 
-      return Promise.all([
-        this.fetchUserPreferredWebexSite(),
-        this.getGeoHint(),
-        this.startReachability().catch((error) => {
-          LoggerProxy.logger.error(`Meetings:index#register --> GDM error, ${error.message}`);
-        }),
-        this.webex.internal.device.register()
-          .then(() => LoggerProxy.logger.info(`Meetings:index#register --> INFO, Device registered ${this.webex.internal.device.url}`))
-          .then(() => this.webex.internal.mercury.connect()),
-        MeetingsUtil.checkH264Support.call(this)
-      ]).then(() => {
-        this.listenForEvents();
-        Trigger.trigger(
-          this,
-          {
-            file: 'meetings',
-            function: 'register'
-          },
-          EVENT_TRIGGERS.MEETINGS_REGISTERED
-        );
-        this.registered = true;
         Metrics.sendBehavioralMetric(
-          BEHAVIORAL_METRICS.MEETINGS_REGISTRATION_SUCCESS,
+          BEHAVIORAL_METRICS.MEETINGS_REGISTRATION_FAILED,
+          {
+            reason: error.message,
+            stack: error.stack
+          }
         );
-      })
-        .catch((error) => {
-          LoggerProxy.logger.error(`Meetings:index#register --> ERROR, Unable to register, ${error.message}`);
-
-          Metrics.sendBehavioralMetric(
-            BEHAVIORAL_METRICS.MEETINGS_REGISTRATION_FAILED,
-            {
-              reason: error.message,
-              stack: error.stack
-            }
-          );
 
-          return Promise.reject(error);
-        });
-    }
+        return Promise.reject(error);
+      });
+  }
 
-    /**
+  /**
      * Explicitly tears down the meetings plugin by deregistering
      * the device, disconnecting from mercury, and stops listening to locus events
      *
@@ -503,31 +500,31 @@ export default class Meetings extends WebexPlugin {
      * @public
      * @memberof Meetings
      */
-    unregister() {
-      if (!this.registered) {
-        LoggerProxy.logger.info('Meetings:index#unregister --> INFO, Meetings plugin already unregistered');
+  unregister() {
+    if (!this.registered) {
+      LoggerProxy.logger.info('Meetings:index#unregister --> INFO, Meetings plugin already unregistered');
 
-        return Promise.resolve();
-      }
-
-      this.stopListeningForEvents();
-
-      return this.webex.internal.mercury.disconnect()
-        .then(() => this.webex.internal.device.unregister())
-        .then(() => {
-          Trigger.trigger(
-            this,
-            {
-              file: 'meetings',
-              function: 'unregister'
-            },
-            EVENT_TRIGGERS.MEETINGS_UNREGISTERED
-          );
-          this.registered = false;
-        });
+      return Promise.resolve();
     }
 
-    /**
+    this.stopListeningForEvents();
+
+    return this.webex.internal.mercury.disconnect()
+      .then(() => this.webex.internal.device.unregister())
+      .then(() => {
+        Trigger.trigger(
+          this,
+          {
+            file: 'meetings',
+            function: 'unregister'
+          },
+          EVENT_TRIGGERS.MEETINGS_UNREGISTERED
+        );
+        this.registered = false;
+      });
+  }
+
+  /**
      * Uploads logs to the webex services for tracking
      * @param {Object} [options={}]
      * @param {String} [options.callStart] Call Start Time
@@ -539,126 +536,126 @@ export default class Meetings extends WebexPlugin {
      * @param {String} [options.orgId] org id
      * @returns {String} feedback ID logs were submitted under
      */
-    uploadLogs(options = {}) {
-      LoggerProxy.logger.info('Meetings:index#uploadLogs --> uploading logs');
-
-      return this.loggerRequest.uploadLogs(options)
-        .then((uploadResult) => {
-          LoggerProxy.logger.info('Meetings:index#uploadLogs --> Upload logs for meeting completed.', uploadResult);
-          Trigger.trigger(
-            this,
-            {
-              file: 'meetings',
-              function: 'uploadLogs'
-            },
-            EVENT_TRIGGERS.MEETING_LOG_UPLOAD_SUCCESS,
-            {
-              meetingId: options.meetingId,
-              details: uploadResult
-            }
-          );
-        })
-        .catch((uploadError) => {
-          LoggerProxy.logger.error('Meetings:index#uploadLogs --> Unable to upload logs for meeting', uploadError);
-          Trigger.trigger(
-            this,
-            {
-              file: 'meetings',
-              function: 'uploadLogs'
-            },
-            EVENT_TRIGGERS.MEETING_LOG_UPLOAD_FAILURE,
-            {
-              meetingId: options.meetingId,
-              reason: uploadError
-            }
-          );
-
-          Metrics.sendBehavioralMetric(
-            BEHAVIORAL_METRICS.UPLOAD_LOGS_FAILURE,
-            {
-              meetingId: options.meetingsId,
-              reason: uploadError.message,
-              stack: uploadError.stack,
-              code: uploadError.code
-            }
-          );
-        });
-    }
+  uploadLogs(options = {}) {
+    LoggerProxy.logger.info('Meetings:index#uploadLogs --> uploading logs');
 
-    /**
+    return this.loggerRequest.uploadLogs(options)
+      .then((uploadResult) => {
+        LoggerProxy.logger.info('Meetings:index#uploadLogs --> Upload logs for meeting completed.', uploadResult);
+        Trigger.trigger(
+          this,
+          {
+            file: 'meetings',
+            function: 'uploadLogs'
+          },
+          EVENT_TRIGGERS.MEETING_LOG_UPLOAD_SUCCESS,
+          {
+            meetingId: options.meetingId,
+            details: uploadResult
+          }
+        );
+      })
+      .catch((uploadError) => {
+        LoggerProxy.logger.error('Meetings:index#uploadLogs --> Unable to upload logs for meeting', uploadError);
+        Trigger.trigger(
+          this,
+          {
+            file: 'meetings',
+            function: 'uploadLogs'
+          },
+          EVENT_TRIGGERS.MEETING_LOG_UPLOAD_FAILURE,
+          {
+            meetingId: options.meetingId,
+            reason: uploadError
+          }
+        );
+
+        Metrics.sendBehavioralMetric(
+          BEHAVIORAL_METRICS.UPLOAD_LOGS_FAILURE,
+          {
+            meetingId: options.meetingsId,
+            reason: uploadError.message,
+            stack: uploadError.stack,
+            code: uploadError.code
+          }
+        );
+      });
+  }
+
+  /**
      * initializes the reachability instance for Meetings
      * @returns {undefined}
      * @public
      * @memberof Meetings
      */
-    setReachability() {
-      this.reachability = new Reachability(this.webex);
-    }
+  setReachability() {
+    this.reachability = new Reachability(this.webex);
+  }
 
-    /**
+  /**
      * gets the reachability instance for Meetings
      * @returns {Reachability}
      * @public
      * @memberof Meetings
      */
-    getReachability() {
-      return this.reachability;
-    }
+  getReachability() {
+    return this.reachability;
+  }
 
-    /**
+  /**
      * initializes and starts gathering reachability for Meetings
      * @returns {Promise}
      * @public
      * @memberof Meetings
      */
-    startReachability() {
-      if (!this.reachability) {
-        this.setReachability();
-      }
-
-      return this.getReachability().gatherReachability();
+  startReachability() {
+    if (!this.reachability) {
+      this.setReachability();
     }
 
-    /**
+    return this.getReachability().gatherReachability();
+  }
+
+  /**
      * Get geoHint for info for meetings
      * @returns {Promise}
      * @private
      * @memberof Meetings
      */
-    getGeoHint() {
-      return this.request.fetchGeoHint().then((res) => {
-        this.geoHintInfo = res;
-      });
-    }
+  getGeoHint() {
+    return this.request.fetchGeoHint().then((res) => {
+      this.geoHintInfo = res;
+    });
+  }
 
-    /**
+  /**
      * Fetch user preferred webex site information
      * This also has other infomation about the user
      * @returns {Promise}
      * @private
      * @memberof Meetings
      */
-    fetchUserPreferredWebexSite() {
-      return this.request.fetchLoginUserInformation().then((res) => {
-        if (res && res.userPreferences) {
-          this.preferredWebexSite = MeetingsUtil.parseUserPreferences(res?.userPreferences);
-        }
-      });
-    }
+  fetchUserPreferredWebexSite() {
+    return this.request.fetchLoginUserInformation().then((res) => {
+      if (res && res.userPreferences) {
+        this.preferredWebexSite = MeetingsUtil.parseUserPreferences(res?.userPreferences);
+      }
+    });
+  }
 
 
-    /**
+  /**
      * gets the personal meeting room instance, for saved PMR values for this user
      * @returns {PersonalMeetingRoom}
      * @public
      * @memberof Meetings
      */
 
-    getPersonalMeetingRoom() {
-      return this.personalMeetingRoom;
-    }
+  getPersonalMeetingRoom() {
+    return this.personalMeetingRoom;
+  }
 
-    /**
+  /**
      * @param {Meeting} meeting
      * @param {Object} reason
      * @param {String} type
@@ -666,24 +663,24 @@ export default class Meetings extends WebexPlugin {
      * @private
      * @memberof Meetings
      */
-    destroy(meeting, reason) {
-      MeetingUtil.cleanUp(meeting);
-      this.meetingCollection.delete(meeting.id);
-      Trigger.trigger(
-        this,
-        {
-          file: 'meetings',
-          function: 'destroy'
-        },
-        EVENT_TRIGGERS.MEETING_REMOVED,
-        {
-          meetingId: meeting.id,
-          reason
-        }
-      );
-    }
+  destroy(meeting, reason) {
+    MeetingUtil.cleanUp(meeting);
+    this.meetingCollection.delete(meeting.id);
+    Trigger.trigger(
+      this,
+      {
+        file: 'meetings',
+        function: 'destroy'
+      },
+      EVENT_TRIGGERS.MEETING_REMOVED,
+      {
+        meetingId: meeting.id,
+        reason
+      }
+    );
+  }
 
-    /**
+  /**
      * Create a meeting.
      * @param {string} destination - sipURL, spaceId, phonenumber, or locus object}
      * @param {string} [type] - the optional specified type, such as locusId
@@ -691,182 +688,182 @@ export default class Meetings extends WebexPlugin {
      * @public
      * @memberof Meetings
      */
-    create(destination, type = null) {
-      // TODO: type should be from a dictionary
-
-      // Validate meeting information based on the provided destination and
-      // type. This must be performed prior to determining if the meeting is
-      // found in the collection, as we mutate the destination for hydra person
-      // id values.
-      return this.meetingInfo.fetchInfoOptions(destination, type)
-        // Catch a failure to fetch info options.
-        .catch((error) => {
-          LoggerProxy.logger.info(`Meetings:index#create --> INFO, unable to determine info options: ${error.message}`);
-        })
-        .then((options = {}) => {
-          // Normalize the destination.
-          const targetDest = options.destination || destination;
-
-          // check for the conversation URL then sip Url
-          let meeting = null;
-
-          if (type === _CONVERSATION_URL_ || options.type === _CONVERSATION_URL_) {
-            const foundMeeting = this.meetingCollection.getByKey(CONVERSATION_URL, targetDest);
-
-            if (foundMeeting) {
-              const foundMeetingIsNotCalendarMeeting = !foundMeeting.locusInfo.scheduledMeeting;
-
-              // If the found meeting is not a calendar meeting, return that meeting.
-              // This allows for the creation of instant-meetings when calendar meetings are present.
-              if (foundMeetingIsNotCalendarMeeting) {
-                meeting = foundMeeting;
-              }
+  create(destination, type = null) {
+    // TODO: type should be from a dictionary
+
+    // Validate meeting information based on the provided destination and
+    // type. This must be performed prior to determining if the meeting is
+    // found in the collection, as we mutate the destination for hydra person
+    // id values.
+    return this.meetingInfo.fetchInfoOptions(destination, type)
+    // Catch a failure to fetch info options.
+      .catch((error) => {
+        LoggerProxy.logger.info(`Meetings:index#create --> INFO, unable to determine info options: ${error.message}`);
+      })
+      .then((options = {}) => {
+        // Normalize the destination.
+        const targetDest = options.destination || destination;
+
+        // check for the conversation URL then sip Url
+        let meeting = null;
+
+        if (type === _CONVERSATION_URL_ || options.type === _CONVERSATION_URL_) {
+          const foundMeeting = this.meetingCollection.getByKey(CONVERSATION_URL, targetDest);
+
+          if (foundMeeting) {
+            const foundMeetingIsNotCalendarMeeting = !foundMeeting.locusInfo.scheduledMeeting;
+
+            // If the found meeting is not a calendar meeting, return that meeting.
+            // This allows for the creation of instant-meetings when calendar meetings are present.
+            if (foundMeetingIsNotCalendarMeeting) {
+              meeting = foundMeeting;
             }
           }
+        }
 
-          // Attempt to collect the meeting if it exists.
-          if (!meeting) {
-            meeting = this.meetingCollection.getByKey(SIP_URI, targetDest);
-          }
+        // Attempt to collect the meeting if it exists.
+        if (!meeting) {
+          meeting = this.meetingCollection.getByKey(SIP_URI, targetDest);
+        }
 
 
-          // Validate if a meeting was found.
-          if (!meeting) {
-            // Create a meeting based on the normalized destination and type.
-            return this.createMeeting(targetDest, type)
-              .then((createdMeeting) => {
-                // If the meeting was successfully created.
-                if (createdMeeting && createdMeeting.on) {
-                  // Create a destruction event for the meeting.
-                  createdMeeting.on(EVENTS.DESTROY_MEETING, (payload) => {
-                    if (this.config.autoUploadLogs) {
-                      this.uploadLogs({
-                        callStart: createdMeeting.locusInfo?.fullState?.lastActive,
-                        correlationId: createdMeeting.correlationId,
-                        feedbackId: createdMeeting.correlationId,
-                        locusId: createdMeeting.locusId,
-                        meetingId: createdMeeting.locusInfo?.info?.webExMeetingId
-                      }).then(() => this.destroy(createdMeeting, payload.reason));
-                    }
-                    else {
-                      this.destroy(createdMeeting, payload.reason);
-                    }
-                  });
-
-                  createdMeeting.on(EVENTS.REQUEST_UPLOAD_LOGS, (meetingInstance) => {
-                    if (this.config.autoUploadLogs) {
-                      this.uploadLogs({
-                        callStart: meetingInstance?.locusInfo?.fullState?.lastActive,
-                        correlationId: meetingInstance.correlationId,
-                        feedbackId: meetingInstance.correlationId,
-                        locusId: meetingInstance.locusId,
-                        meetingId: meetingInstance.locusInfo?.info?.webExMeetingId
-                      });
-                    }
-                  });
-                }
-                else {
-                  LoggerProxy.logger.error(`Meetings:index#create --> ERROR, meeting does not have on method, will not be destroyed, meeting cleanup impossible for meeting: ${meeting}`);
-                }
-
-                // Return the newly created meeting.
-                return Promise.resolve(createdMeeting);
-              });
-          }
+        // Validate if a meeting was found.
+        if (!meeting) {
+          // Create a meeting based on the normalized destination and type.
+          return this.createMeeting(targetDest, type)
+            .then((createdMeeting) => {
+              // If the meeting was successfully created.
+              if (createdMeeting && createdMeeting.on) {
+                // Create a destruction event for the meeting.
+                createdMeeting.on(EVENTS.DESTROY_MEETING, (payload) => {
+                  if (this.config.autoUploadLogs) {
+                    this.uploadLogs({
+                      callStart: createdMeeting.locusInfo?.fullState?.lastActive,
+                      correlationId: createdMeeting.correlationId,
+                      feedbackId: createdMeeting.correlationId,
+                      locusId: createdMeeting.locusId,
+                      meetingId: createdMeeting.locusInfo?.info?.webExMeetingId
+                    }).then(() => this.destroy(createdMeeting, payload.reason));
+                  }
+                  else {
+                    this.destroy(createdMeeting, payload.reason);
+                  }
+                });
+
+                createdMeeting.on(EVENTS.REQUEST_UPLOAD_LOGS, (meetingInstance) => {
+                  if (this.config.autoUploadLogs) {
+                    this.uploadLogs({
+                      callStart: meetingInstance?.locusInfo?.fullState?.lastActive,
+                      correlationId: meetingInstance.correlationId,
+                      feedbackId: meetingInstance.correlationId,
+                      locusId: meetingInstance.locusId,
+                      meetingId: meetingInstance.locusInfo?.info?.webExMeetingId
+                    });
+                  }
+                });
+              }
+              else {
+                LoggerProxy.logger.error(`Meetings:index#create --> ERROR, meeting does not have on method, will not be destroyed, meeting cleanup impossible for meeting: ${meeting}`);
+              }
 
-          // Return the existing meeting.
-          return Promise.resolve(meeting);
-        });
-    }
+              // Return the newly created meeting.
+              return Promise.resolve(createdMeeting);
+            });
+        }
 
-    /**
+        // Return the existing meeting.
+        return Promise.resolve(meeting);
+      });
+  }
+
+  /**
      * @param {String} destination see create()
      * @param {String} type see create()
      * @returns {Promise} a new meeting instance complete with meeting info and destination
      * @private
      * @memberof Meetings
      */
-    async createMeeting(destination, type = null) {
-      const meeting = new Meeting(
-        {
-          userId: this.webex.internal.device.userId,
-          deviceUrl: this.webex.internal.device.url,
-          orgId: this.webex.internal.device.orgId,
-          roapSeq: 0,
-          locus: type === _LOCUS_ID_ ? destination : null, // pass the locus object if present
-          meetingInfoProvider: this.meetingInfo,
-          destination,
-          destinationType: type,
-        },
-        {
-          parent: this.webex
-        }
-      );
+  async createMeeting(destination, type = null) {
+    const meeting = new Meeting(
+      {
+        userId: this.webex.internal.device.userId,
+        deviceUrl: this.webex.internal.device.url,
+        orgId: this.webex.internal.device.orgId,
+        roapSeq: 0,
+        locus: type === _LOCUS_ID_ ? destination : null, // pass the locus object if present
+        meetingInfoProvider: this.meetingInfo,
+        destination,
+        destinationType: type,
+      },
+      {
+        parent: this.webex
+      }
+    );
 
-      this.meetingCollection.set(meeting);
+    this.meetingCollection.set(meeting);
 
-      try {
-        await meeting.fetchMeetingInfo({});
+    try {
+      await meeting.fetchMeetingInfo({});
+    }
+    catch (err) {
+      if (!(err instanceof CaptchaError) && !(err instanceof PasswordError)) {
+        // if there is no meeting info we assume its a 1:1 call or wireless share
+        LoggerProxy.logger.info(`Meetings:index#createMeeting --> Info Unable to fetch meeting info for ${destination}.`);
+        LoggerProxy.logger.info('Meetings:index#createMeeting --> Info assuming this destination is a 1:1 or wireless share');
       }
-      catch (err) {
-        if (!(err instanceof CaptchaError) && !(err instanceof PasswordError)) {
-          // if there is no meeting info we assume its a 1:1 call or wireless share
-          LoggerProxy.logger.info(`Meetings:index#createMeeting --> Info Unable to fetch meeting info for ${destination}.`);
-          LoggerProxy.logger.info('Meetings:index#createMeeting --> Info assuming this destination is a 1:1 or wireless share');
+      LoggerProxy.logger.debug(`Meetings:index#createMeeting --> Debug ${err} fetching /meetingInfo for creation.`);
+    }
+    finally {
+      // For type LOCUS_ID we need to parse the locus object to get the information
+      // about the caller and callee
+      // Meeting Added event will be created in `handleLocusEvent`
+      if (type !== _LOCUS_ID_) {
+        if (!meeting.sipUri) {
+          meeting.setSipUri(destination);
         }
-        LoggerProxy.logger.debug(`Meetings:index#createMeeting --> Debug ${err} fetching /meetingInfo for creation.`);
-      }
-      finally {
-        // For type LOCUS_ID we need to parse the locus object to get the information
-        // about the caller and callee
-        // Meeting Added event will be created in `handleLocusEvent`
-        if (type !== _LOCUS_ID_) {
-          if (!meeting.sipUri) {
-            meeting.setSipUri(destination);
-          }
 
-          // TODO: check if we have to move this to parser
-          const meetingAddedType = MeetingsUtil.getMeetingAddedType(type);
-
-          // We typically shouldn't need to trigger both and event and return a promise.
-          // Is this a special case? We want to make the public API usage as simple as possible.
-          Trigger.trigger(
-            this,
-            {
-              file: 'meetings',
-              function: 'createMeeting'
-            },
-            EVENT_TRIGGERS.MEETING_ADDED,
-            {
-              meeting,
-              type: meetingAddedType
-            }
-          );
-        }
-      }
+        // TODO: check if we have to move this to parser
+        const meetingAddedType = MeetingsUtil.getMeetingAddedType(type);
 
-      return meeting;
-
-      // Create the meeting calling the necessary service endpoints.
-
-      // Internally, there are many more destinations:
-      //
-      // - locusID
-      // - meetingURL
-      // - globalMeetingID, e.g, *00*meetingID
-      // - meetingID
-      // - meetingURL
-      // - PSTN
-      // - phone number
-      //
-      // Our job is to determine the appropriate one
-      // and its corresponding service so that developers
-      // need only sipURL or spaceID to get a meeting
-      // and its ID, but have the option to use createWithType()
-      // and specify those types to get meetingInfo
+        // We typically shouldn't need to trigger both and event and return a promise.
+        // Is this a special case? We want to make the public API usage as simple as possible.
+        Trigger.trigger(
+          this,
+          {
+            file: 'meetings',
+            function: 'createMeeting'
+          },
+          EVENT_TRIGGERS.MEETING_ADDED,
+          {
+            meeting,
+            type: meetingAddedType
+          }
+        );
+      }
     }
 
-    /**
+    return meeting;
+
+    // Create the meeting calling the necessary service endpoints.
+
+    // Internally, there are many more destinations:
+    //
+    // - locusID
+    // - meetingURL
+    // - globalMeetingID, e.g, *00*meetingID
+    // - meetingID
+    // - meetingURL
+    // - PSTN
+    // - phone number
+    //
+    // Our job is to determine the appropriate one
+    // and its corresponding service so that developers
+    // need only sipURL or spaceID to get a meeting
+    // and its ID, but have the option to use createWithType()
+    // and specify those types to get meetingInfo
+  }
+
+  /**
      * get a specifc meeting given it's type matched to the value, i.e., locus url
      * @param {String} type
      * @param {Object} value
@@ -874,11 +871,11 @@ export default class Meetings extends WebexPlugin {
      * @public
      * @memberof Meetings
      */
-    getMeetingByType(type, value) {
-      return this.meetingCollection.getByKey(type, value);
-    }
+  getMeetingByType(type, value) {
+    return this.meetingCollection.getByKey(type, value);
+  }
 
-    /**
+  /**
      * Get all meetings.
      * @param {object} options
      * @param {object} options.startDate - get meetings after this start date
@@ -887,47 +884,47 @@ export default class Meetings extends WebexPlugin {
      * @public
      * @memberof Meetings
      */
-    getAllMeetings(options = {}) {
-      // Options may include other parameters to filter this collection
-      // of meetings.
-      return this.meetingCollection.getAll(options);
-    }
+  getAllMeetings(options = {}) {
+    // Options may include other parameters to filter this collection
+    // of meetings.
+    return this.meetingCollection.getAll(options);
+  }
 
-    /**
+  /**
      * syncs all the meeting from server
      * @returns {undefined}
      * @public
      * @memberof Meetings
      */
-    syncMeetings() {
-      return this.request.getActiveMeetings().then((locusArray) => {
-        const activeLocusUrl = [];
-
-        if (locusArray?.loci && locusArray.loci.length > 0) {
-          locusArray.loci.forEach((locus) => {
-            activeLocusUrl.push(locus.url);
-            this.handleLocusEvent({
-              locus,
-              locusUrl: locus.url
-            });
+  syncMeetings() {
+    return this.request.getActiveMeetings().then((locusArray) => {
+      const activeLocusUrl = [];
+
+      if (locusArray?.loci && locusArray.loci.length > 0) {
+        locusArray.loci.forEach((locus) => {
+          activeLocusUrl.push(locus.url);
+          this.handleLocusEvent({
+            locus,
+            locusUrl: locus.url
           });
-        }
-        const meetingsCollection = this.meetingCollection.getAll();
-
-        if (Object.keys(meetingsCollection).length > 0) {
-          // Some time the mercury event is missed after mercury reconnect
-          // if sync returns no locus then clear all the meetings
-          for (const meeting of Object.values(meetingsCollection)) {
-            if (!activeLocusUrl.includes(meeting.locusUrl)) {
-              // destroy function also uploads logs
-              this.destroy(meeting, MEETING_REMOVED_REASON.NO_MEETINGS_TO_SYNC);
-            }
+        });
+      }
+      const meetingsCollection = this.meetingCollection.getAll();
+
+      if (Object.keys(meetingsCollection).length > 0) {
+        // Some time the mercury event is missed after mercury reconnect
+        // if sync returns no locus then clear all the meetings
+        for (const meeting of Object.values(meetingsCollection)) {
+          if (!activeLocusUrl.includes(meeting.locusUrl)) {
+            // destroy function also uploads logs
+            this.destroy(meeting, MEETING_REMOVED_REASON.NO_MEETINGS_TO_SYNC);
           }
         }
-      });
-    }
+      }
+    });
+  }
 
-    /**
+  /**
      * Get all scheduled meetings.
      * @param {object} options
      * @param {object} options.startDate - get meetings after this start date
@@ -935,35 +932,35 @@ export default class Meetings extends WebexPlugin {
      * @returns {Object} All scheduled meetings.
      * @memberof Meetings
      */
-    getScheduledMeetings() {
-      return this.meetingCollection.getAll({scheduled: true});
-    }
+  getScheduledMeetings() {
+    return this.meetingCollection.getAll({scheduled: true});
+  }
 
-    /**
+  /**
      * Get the logger instance for plugin-meetings
      * @returns {Logger}
      */
-    getLogger() {
-      return LoggerProxy.get();
-    }
+  getLogger() {
+    return LoggerProxy.get();
+  }
 
-    /**
+  /**
      * Get the Stats Analyzer singleton
      * @returns {StatsAnalyzer}
      * @public
      * @memberof Meetings
      */
-    getAnalyzer() {
-      return StatsAnalyzer;
-    }
+  getAnalyzer() {
+    return StatsAnalyzer;
+  }
 
-    /**
+  /**
      * Get the Stats Calculator singleton
      * @returns {StatsCalculator}
      * @public
      * @memberof Meetings
      */
-    getCalculator() {
-      return StatsCalculator;
-    }
+  getCalculator() {
+    return StatsCalculator;
+  }
 }