fix: add null check for role change while updating plugins

100mslive · Mar 26, 2024 · 2660667 · 2660667
1 parent fd75519
commit 2660667
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/hms-video-store/src/sdk/store/Store.ts b/packages/hms-video-store/src/sdk/store/Store.ts
@@ -24,12 +24,14 @@ import {
   HMSVideoTrack,
 } from '../../media/tracks';
 import { PolicyParams } from '../../notification-manager';
+import HMSLogger from '../../utils/logger';
 import { ENV } from '../../utils/support';
 import { createUserAgent } from '../../utils/user-agent';
 import HMSRoom from '../models/HMSRoom';
 import { HMSLocalPeer, HMSPeer, HMSRemotePeer } from '../models/peer';
 
 class Store {
+  private TAG = '[Store]:';
   private room?: HMSRoom;
   private knownRoles: KnownRoles = {};
   private localPeerId?: string;
@@ -415,6 +417,10 @@ class Store {
       pluginName: keyof PolicyParams['plugins'],
       permission: HMSPermissionType,
     ) => {
+      if (!this.knownRoles[role]) {
+        HMSLogger.d(this.TAG, `role ${role} is not present in given roles`, this.knownRoles);
+        return;
+      }
       const rolePermissions = this.knownRoles[role].permissions;
       if (!rolePermissions[pluginName]) {
         rolePermissions[pluginName] = [];