@webex/plugin-meetings 3.0.0-beta.180 → 3.0.0-beta.182

package/dist/types/locus-info/parser.d.ts

@@ -80,6 +80,21 @@ export default class Parser {
      * @returns {string} loci comparison state
      */
     private static compareDelta;
+    /**
+     * Compares Locus sequences - it should be called only for full Locus DTOs, not deltas
+     *
+     * @param {Types~Locus} current Current working copy
+     * @param {Types~Locus} incomingFullDto New Full Locus DTO
+     * @returns {string} either Parser.loci.USE_INCOMING or Parser.loci.USE_CURRENT
+     */
+    static compareFullDtoSequence(current: any, incomingFullDto: any): string;
+    /**
+     * Returns true if the incoming full locus DTO is newer than the current working copy
+     *
+     * @param {Types~Locus} incomingFullDto New Full Locus DTO
+     * @returns {string} either Parser.loci.USE_INCOMING or Parser.loci.USE_CURRENT
+     */
+    isNewFullLocus(incomingFullDto: any): boolean;
     /**
      * Compares Locus sequences
      * @param {Types~Locus} current Current working copy

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@webex/plugin-meetings",
-  "version": "3.0.0-beta.180",
+  "version": "3.0.0-beta.182",
   "description": "",
   "license": "Cisco EULA (https://www.cisco.com/c/en/us/products/end-user-license-agreement.html)",
   "contributors": [
@@ -32,12 +32,12 @@
     "build": "yarn run -T tsc --declaration true --declarationDir ./dist/types"
   },
   "devDependencies": {
-    "@webex/plugin-meetings": "3.0.0-beta.180",
-    "@webex/test-helper-chai": "3.0.0-beta.180",
-    "@webex/test-helper-mocha": "3.0.0-beta.180",
-    "@webex/test-helper-mock-webex": "3.0.0-beta.180",
-    "@webex/test-helper-retry": "3.0.0-beta.180",
-    "@webex/test-helper-test-users": "3.0.0-beta.180",
+    "@webex/plugin-meetings": "3.0.0-beta.182",
+    "@webex/test-helper-chai": "3.0.0-beta.182",
+    "@webex/test-helper-mocha": "3.0.0-beta.182",
+    "@webex/test-helper-mock-webex": "3.0.0-beta.182",
+    "@webex/test-helper-retry": "3.0.0-beta.182",
+    "@webex/test-helper-test-users": "3.0.0-beta.182",
     "chai": "^4.3.4",
     "chai-as-promised": "^7.1.1",
     "jsdom-global": "3.0.2",
@@ -46,19 +46,19 @@
     "typescript": "^4.7.4"
   },
   "dependencies": {
-    "@webex/common": "3.0.0-beta.180",
+    "@webex/common": "3.0.0-beta.182",
     "@webex/internal-media-core": "1.39.1",
-    "@webex/internal-plugin-conversation": "3.0.0-beta.180",
-    "@webex/internal-plugin-device": "3.0.0-beta.180",
-    "@webex/internal-plugin-llm": "3.0.0-beta.180",
-    "@webex/internal-plugin-mercury": "3.0.0-beta.180",
-    "@webex/internal-plugin-metrics": "3.0.0-beta.180",
-    "@webex/internal-plugin-support": "3.0.0-beta.180",
-    "@webex/internal-plugin-user": "3.0.0-beta.180",
-    "@webex/media-helpers": "3.0.0-beta.180",
-    "@webex/plugin-people": "3.0.0-beta.180",
-    "@webex/plugin-rooms": "3.0.0-beta.180",
-    "@webex/webex-core": "3.0.0-beta.180",
+    "@webex/internal-plugin-conversation": "3.0.0-beta.182",
+    "@webex/internal-plugin-device": "3.0.0-beta.182",
+    "@webex/internal-plugin-llm": "3.0.0-beta.182",
+    "@webex/internal-plugin-mercury": "3.0.0-beta.182",
+    "@webex/internal-plugin-metrics": "3.0.0-beta.182",
+    "@webex/internal-plugin-support": "3.0.0-beta.182",
+    "@webex/internal-plugin-user": "3.0.0-beta.182",
+    "@webex/media-helpers": "3.0.0-beta.182",
+    "@webex/plugin-people": "3.0.0-beta.182",
+    "@webex/plugin-rooms": "3.0.0-beta.182",
+    "@webex/webex-core": "3.0.0-beta.182",
     "ampersand-collection": "^2.0.2",
     "bowser": "^2.11.0",
     "btoa": "^1.2.1",

package/src/controls-options-manager/util.ts

@@ -89,6 +89,20 @@ class Utils {
     return requiredHints.every((hint) => displayHints.includes(hint));
   }
 
+  /**
+   * Validate that the self policy object contains the required policies.
+   *
+   * @param {Object} config - Configuration Object.
+   * @param {Array<string>} config.requiredPolicies - Policies required for validation.
+   * @param {Array<string>} config.policies - All available policies.
+   * @returns {boolean} - True if all of the actions are allowed.
+   */
+  public static hasPolicies(config: {requiredPolicies: Array<string>; policies: Array<string>}) {
+    const {requiredPolicies, policies = {}} = config;
+
+    return requiredPolicies.every((hint) => policies[hint]);
+  }
+
   /**
    * Validate if an audio-scoped control is allowed to be sent to the service.
    *

package/src/locus-info/index.ts

@@ -267,13 +267,20 @@ export default class LocusInfo extends EventsScope {
    * @returns {object} null
    * @memberof LocusInfo
    */
-  // eslint-disable-next-line @typescript-eslint/no-shadow
   onFullLocus(locus: any, eventType?: string) {
     if (!locus) {
       LoggerProxy.logger.error(
         'Locus-info:index#onFullLocus --> object passed as argument was invalid, continuing.'
       );
     }
+
+    if (!this.locusParser.isNewFullLocus(locus)) {
+      LoggerProxy.logger.info(
+        `Locus-info:index#onFullLocus --> ignoring old full locus DTO, eventType=${eventType}`
+      );
+
+      return;
+    }
     this.updateParticipantDeltas(locus.participants);
     this.scheduledMeeting = locus.meeting || null;
     this.participants = locus.participants;

package/src/locus-info/parser.ts

@@ -235,6 +235,49 @@ export default class Parser {
     return pack(comparison, result);
   }
 
+  /**
+   * Compares Locus sequences - it should be called only for full Locus DTOs, not deltas
+   *
+   * @param {Types~Locus} current Current working copy
+   * @param {Types~Locus} incomingFullDto New Full Locus DTO
+   * @returns {string} either Parser.loci.USE_INCOMING or Parser.loci.USE_CURRENT
+   */
+  static compareFullDtoSequence(current, incomingFullDto) {
+    if (Parser.isSequenceEmpty(current) || Parser.isSequenceEmpty(incomingFullDto)) {
+      return Parser.loci.USE_INCOMING;
+    }
+
+    // the sequence.entries list will always contain at least 1 entry
+    // https://sqbu-github.cisco.com/WebExSquared/cloud-apps/wiki/Locus-Sequence-Comparison-Algorithm
+
+    return incomingFullDto.sequence.entries.slice(-1)[0] > current.sequence.entries.slice(-1)[0]
+      ? Parser.loci.USE_INCOMING
+      : Parser.loci.USE_CURRENT;
+  }
+
+  /**
+   * Returns true if the incoming full locus DTO is newer than the current working copy
+   *
+   * @param {Types~Locus} incomingFullDto New Full Locus DTO
+   * @returns {string} either Parser.loci.USE_INCOMING or Parser.loci.USE_CURRENT
+   */
+  isNewFullLocus(incomingFullDto) {
+    if (!Parser.isLoci(incomingFullDto)) {
+      LoggerProxy.logger.info('Locus-info:parser#isNewFullLocus --> Ignoring non-locus object.');
+
+      return false;
+    }
+
+    if (!this.workingCopy) {
+      // we don't have a working copy yet, so any full locus is better than nothing
+      return true;
+    }
+
+    const comparisonResult = Parser.compareFullDtoSequence(this.workingCopy, incomingFullDto);
+
+    return comparisonResult === Parser.loci.USE_INCOMING;
+  }
+
   /**
    * Compares Locus sequences
    * @param {Types~Locus} current Current working copy

package/src/meeting/index.ts

@@ -90,6 +90,7 @@ import {
   HTTP_VERBS,
   SELF_ROLES,
   INTERPRETATION,
+  SELF_POLICY,
 } from '../constants';
 import BEHAVIORAL_METRICS from '../metrics/constants';
 import ParameterError from '../common/errors/parameter';
@@ -2585,22 +2586,42 @@ export default class Meeting extends StatelessWebexPlugin {
             requiredHints: [DISPLAY_HINTS.DISABLE_VIDEO],
             displayHints: payload.info.userDisplayHints,
           }),
-          canShareFile: ControlsOptionsUtil.hasHints({
-            requiredHints: [DISPLAY_HINTS.SHARE_FILE],
-            displayHints: payload.info.userDisplayHints,
-          }),
-          canShareApplication: ControlsOptionsUtil.hasHints({
-            requiredHints: [DISPLAY_HINTS.SHARE_APPLICATION],
-            displayHints: payload.info.userDisplayHints,
-          }),
-          canShareCamera: ControlsOptionsUtil.hasHints({
-            requiredHints: [DISPLAY_HINTS.SHARE_CAMERA],
-            displayHints: payload.info.userDisplayHints,
-          }),
-          canShareDesktop: ControlsOptionsUtil.hasHints({
-            requiredHints: [DISPLAY_HINTS.SHARE_DESKTOP],
-            displayHints: payload.info.userDisplayHints,
-          }),
+          canShareFile:
+            ControlsOptionsUtil.hasHints({
+              requiredHints: [DISPLAY_HINTS.SHARE_FILE],
+              displayHints: payload.info.userDisplayHints,
+            }) &&
+            ControlsOptionsUtil.hasPolicies({
+              requiredPolicies: [SELF_POLICY.SUPPORT_FILE_SHARE],
+              policies: this.selfUserPolicies,
+            }),
+          canShareApplication:
+            ControlsOptionsUtil.hasHints({
+              requiredHints: [DISPLAY_HINTS.SHARE_APPLICATION],
+              displayHints: payload.info.userDisplayHints,
+            }) &&
+            ControlsOptionsUtil.hasPolicies({
+              requiredPolicies: [SELF_POLICY.SUPPORT_APP_SHARE],
+              policies: this.selfUserPolicies,
+            }),
+          canShareCamera:
+            ControlsOptionsUtil.hasHints({
+              requiredHints: [DISPLAY_HINTS.SHARE_CAMERA],
+              displayHints: payload.info.userDisplayHints,
+            }) &&
+            ControlsOptionsUtil.hasPolicies({
+              requiredPolicies: [SELF_POLICY.SUPPORT_CAMERA_SHARE],
+              policies: this.selfUserPolicies,
+            }),
+          canShareDesktop:
+            ControlsOptionsUtil.hasHints({
+              requiredHints: [DISPLAY_HINTS.SHARE_DESKTOP],
+              displayHints: payload.info.userDisplayHints,
+            }) &&
+            ControlsOptionsUtil.hasPolicies({
+              requiredPolicies: [SELF_POLICY.SUPPORT_DESKTOP_SHARE],
+              policies: this.selfUserPolicies,
+            }),
           canShareContent: ControlsOptionsUtil.hasHints({
             requiredHints: [DISPLAY_HINTS.SHARE_CONTENT],
             displayHints: payload.info.userDisplayHints,