fix: Ensure iframe load listener is removed

FrederikBolding · FrederikBolding · commit 56dce3b66487 · 2026-01-09T15:15:27.000+01:00
diff --git a/packages/snaps-utils/src/iframe.ts b/packages/snaps-utils/src/iframe.ts
@@ -50,18 +50,22 @@ export async function createWindow({
     iframe.setAttribute('src', uri);
     document.body.appendChild(iframe);
 
-    iframe.addEventListener('load', () => {
-      if (iframe.contentWindow) {
-        resolve(iframe.contentWindow);
-      } else {
-        // We don't know of a case when this would happen, but better to fail
-        // fast if it does.
-        reject(
-          new Error(
-            `iframe.contentWindow not present on load for job "${id}".`,
-          ),
-        );
-      }
-    });
+    iframe.addEventListener(
+      'load',
+      () => {
+        if (iframe.contentWindow) {
+          resolve(iframe.contentWindow);
+        } else {
+          // We don't know of a case when this would happen, but better to fail
+          // fast if it does.
+          reject(
+            new Error(
+              `iframe.contentWindow not present on load for job "${id}".`,
+            ),
+          );
+        }
+      },
+      { once: true },
+    );
   });
 }
