fix: remove message listener when embed iframe is detroyed

1024pix · Jul 2, 2024 · f347a48 · f347a48
1 parent 6ff64a2
commit f347a48
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/mon-pix/app/components/challenge-embed-simulator.js b/mon-pix/app/components/challenge-embed-simulator.js
@@ -14,6 +14,8 @@ export default class ChallengeEmbedSimulator extends Component {
   @tracked
   embedHeight;
 
+  _embedMessageListener;
+
   constructor(owner, args) {
     super(owner, args);
     this.embedHeight = args.embedDocument?.height;
@@ -42,7 +44,7 @@ export default class ChallengeEmbedSimulator extends Component {
 
     iframe.addEventListener('load', loadListener);
 
-    window.addEventListener('message', ({ origin, data }) => {
+    thisComponent._embedMessageListener = ({ origin, data }) => {
       if (!isEmbedAllowedOrigin(origin)) return;
       if (isReadyMessage(data) && thisComponent.isSimulatorLaunched) {
         iframe.contentWindow.postMessage('launch', '*');
@@ -54,7 +56,9 @@ export default class ChallengeEmbedSimulator extends Component {
       if (isAutoLaunchMessage(data)) {
         thisComponent.launchSimulator();
       }
-    });
+    };
+
+    window.addEventListener('message', thisComponent._embedMessageListener);
   }
 
   @action
@@ -87,6 +91,11 @@ export default class ChallengeEmbedSimulator extends Component {
     iframe.src = 'about:blank';
   }
 
+  willDestroy() {
+    super.willDestroy();
+    window.removeEventListener('message', this._embedMessageListener);
+  }
+
   get iframe() {
     return document.querySelector('.embed__iframe');
   }