Merge pull request #21 from Brahma-fi/ft-rabby-fix

Rabby detection fix
Brahma-fi · May 22, 2024 · df5161a · df5161a
2 parents 5736648 + 1a83430
commit df5161a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/injection.ts b/src/injection.ts
@@ -13,11 +13,12 @@ const chainId = chainIdEl.innerHTML
 // inject bridged ethereum provider
 const injectedProvider = new InjectedProvider(chainId)
 
-try {
+const descriptor = Object.getOwnPropertyDescriptor(window, 'ethereum')
+if (descriptor?.configurable) {
   window.ethereum = injectedProvider
   window.web3.currentProvider = injectedProvider
-} catch (e) {
-  console.log('error overriding provider', e)
+} else {
+  console.log('non configurable window.ethereum detected')
   window.ethereum.request = injectedProvider.request
   window.web3.currentProvider.request = injectedProvider.request
 }