fix(): do not block permission request

airgap-it · Oct 13, 2023 · 324be0f · 324be0f
1 parent b35e30f
commit 324be0f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/app/services/beacon.service.ts b/src/app/services/beacon.service.ts
@@ -79,8 +79,11 @@ export class BeaconService {
     });
   }
   async handleMessage(message: BeaconRequestOutputMessage) {
-    if (localStorage.getItem(StorageKeys.METAMASK_BUSY)) {
-      console.log('MetaMask is busy handling another request');
+    if (
+      localStorage.getItem(StorageKeys.METAMASK_BUSY) &&
+      message.type !== BeaconMessageType.PermissionRequest // Permission requests don't show UI, so we let them through
+    ) {
+      console.log('BeaconService: MetaMask is busy handling another request');
       return;
     }
     if (localStorage.getItem(`${StorageKeys.REQUEST_ID_PREFIX}${message.id}`)) {

diff --git a/src/app/services/metamask.service.ts b/src/app/services/metamask.service.ts
@@ -24,7 +24,7 @@ export class MetamaskService {
     console.log('snap installed', installedSnap);
 
     if (localStorage.getItem(StorageKeys.METAMASK_BUSY)) {
-      console.log('MetaMask is busy handling another request');
+      console.log('MetaMaskService: MetaMask is busy handling another request');
       throw new Error('MetaMask is busy handling another request');
     }
     localStorage.setItem(StorageKeys.METAMASK_BUSY, 'true');