fix: add retry/timeout strategy for injected connector initialization

wevm · Dec 7, 2023 · e9080ff · e9080ff
1 parent e8ba4f5
commit e9080ff
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/packages/core/src/connectors/injected.ts b/packages/core/src/connectors/injected.ts
@@ -9,6 +9,8 @@ import {
   UserRejectedRequestError,
   getAddress,
   numberToHex,
+  withRetry,
+  withTimeout,
 } from 'viem'
 
 import { ChainNotConfiguredError } from '../errors/config.js'
@@ -305,7 +307,13 @@ export function injected(parameters: InjectedParameters = {}) {
           throw new ProviderNotFoundError()
         }
 
-        const accounts = await this.getAccounts()
+        // We are applying a retry & timeout strategy here as some injected wallets (ie. MetaMask) fail to
+        // immediately resolve a JSON-RPC request on page load.
+        const accounts = await withRetry(() =>
+          withTimeout(() => this.getAccounts(), {
+            timeout: 100,
+          }),
+        )
         return !!accounts.length
       } catch {
         return false