Fixed an issue with removing message ids

OfficeDev · May 10, 2024 · 7491372 · 7491372
1 parent 2b41e03
commit 7491372
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/teams-js/src/internal/communication.ts b/packages/teams-js/src/internal/communication.ts
@@ -677,6 +677,9 @@ function removeMessageHandlers(message: MessageResponse, map: Map<MessageUUID, F
   }
   if (!message.uuid) {
     delete CommunicationPrivate.legacyMessageIdsToUuidMap[message.id];
+  } else {
+    //If we are here, then the parent is capable of sending UUIDs, therefore free up memory
+    CommunicationPrivate.legacyMessageIdsToUuidMap = {};
   }
 }
 
@@ -727,6 +730,9 @@ function handleParentMessage(evt: DOMMessageEvent): void {
         logger('Removing registered port callback for message %i', callbackId);
         removeMessageHandlers(message, CommunicationPrivate.portCallbacks);
       }
+      if (message.uuid) {
+        CommunicationPrivate.legacyMessageIdsToUuidMap = {};
+      }
     }
   } else if ('func' in evt.data && typeof evt.data.func === 'string') {
     // Delegate the request to the proper handler