[Sentry] Error: [ethjs-query] the Eth object requires that the first input 'provider' must be an object, got 'und...

[sentry-io]

Sentry Issue: METAMASK-MOBILE-29PE

Error: [ethjs-query] the Eth object requires that the first input 'provider' must be an object, got 'undefined' (i.e. 'const eth = new Eth(provider);')
  at r (ses.cjs:7575:17)
  at constructor (app/core/Engine.ts:600:32)
  at default.init (app/core/Engine.ts:1190:39)
  at initalizeEngine (app/core/EngineService/EngineService.ts:32:12)
  at onPersistComplete (app/store/index.ts:61:19)
...
(17 additional frame(s) were not displayed)

[christopherferreira9]

Hi! I can’t see any reference to WalletConnect other than the init that occurs when launching the wallet. Is there any extra info available?

[sethkfman]

Play Console Log

Exception v6.c: Error: [ethjs-query] the Eth object requires that the first input 'provider' must be an object, got 'undefined' (i.e. 'const eth = new Eth(provider);'), js engine: hermes, stack:
Eth@1:2775775
NonceTracker@1:7371352
TransactionController@1:7358673
Engine@1:2709815
init@1:2716810
anonymous@1:5988908
onPersistComplete@1:5708523
rehydrate@1:5776201
_rehydrate@1:5767164
anonymous@1:5767258
anonymous@1:363065
_callTimer@1:362347
callTimers@1:364149
__callFunction@1:266653
anonymous@1:265020
__guard@1:265972
callFunctionReturnFlushedQueue@1:264978
  at com.facebook.react.modules.core.ExceptionsManagerModule.reportException (ExceptionsManagerModule.java:72)
  at java.lang.reflect.Method.invoke
  at com.facebook.react.bridge.JavaMethodWrapper.invoke (JavaMethodWrapper.java:372)
  at com.facebook.react.bridge.JavaModuleWrapper.invoke (JavaModuleWrapper.java:188)
  at com.facebook.jni.NativeRunnable.run (NativeRunnable.java)
  at android.os.Handler.handleCallback (Handler.java:971)
  at android.os.Handler.dispatchMessage (Handler.java:107)
  at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage (MessageQueueThreadHandler.java:27)
  at android.os.Looper.loopOnce (Looper.java:206)
  at android.os.Looper.loop (Looper.java:296)
  at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run (MessageQueueThreadImpl.java:228)
  at java.lang.Thread.run (Thread.java:1012)

[Cal-L]

@christopherferreira9 Digging into a little deeper:

We can see two logs

• WalletConnectV2::init(), which indicates WCV2 is initializing (indicating it's most likely a cold start from a deeplink)
• Error: Engine does not exist, which happens when you try to access Engine.context without it being defined

So far, I can see WCV2 being initialized in Nav/App/index.js, which is a flow that is mounted post rehydration (onPersistComplete). At the same time, onPersistComplete is where Engine is initialized. Both happens in parallel and I suspect we're seeing the issue because WCV2 is initializing faster than the Engine being initialized. You can probably reproduce the same error message by wrapping EngineService.initalizeEngine(store); in a timer.

[abretonc7s]

Hey @Cal-L , I was just looking at it with Chris

For the init flow:

• store initialize
• onPersistComplete is triggered
• Engine initializes via EngineService.initalizeEngine(store);
• ConnectedRoot opens Nav/App/Index.js (after engine has initialized)
• WalletConnectV2::init() is triggered

I don't think it is wallet connect / sdk related.

Any other ideas?

[sentry-io]

Sentry Issue: METAMASK-MOBILE-29SX

[Cal-L]

@abretonc7s @christopherferreira9 Digging into a bit more, we found that to be the case as well. The WC message is just for initialization. I will reassign this to platform for now. Thanks for the investigation!

[gauthierpetetin]

This is probably related to the crash we're experiencing when upgrading the app to 7.17.0

[sentry-io]

Sentry Issue: METAMASK-MOBILE-29X4

[sentry-io]

Sentry Issue: METAMASK-MOBILE-29WC

[sentry-io]

Sentry Issue: METAMASK-MOBILE-2A8V

[sentry-io]

Sentry Issue: METAMASK-MOBILE-2A8C