MetaMask / metamask-sdk

The simplest yet most secure way to connect your blockchain-based applications to millions of MetaMask Wallet users.
https://metamask.io/sdk/
Other
165 stars 101 forks source link

[ReactNative, ios] When re-connect is repeated, there is a problem that does not connect normally (MetaMask SDK Version: v0.2.3) #112

Closed reese-1eco closed 1 year ago

reese-1eco commented 1 year ago

Describe the bug

To Reproduce Steps to reproduce the behaviour:

  1. Run the recatNativeDemo app.
  2. Click the 'connect' button.
  3. Connect a Metamask account and switch to recatNativeDemo.
  4. Display the connect info on recatNativeDemo.
  5. Click the terminate button.
  6. Retry from step 2 (clicking the 'connect' button) onwards -> -> Success or Failure (no specific error message is displayed).

Expected behaviour I expect it to connect successfully to metamask after calling "terminate()" or after re-running "recatNativeDemo".

Screenshots Attach "reconnect-failed.mp4" including the test process.

Provide environment information

Additional context

reese-1eco commented 1 year ago

Further explanation of the operation of the video attached above (reconnect-failed.mp4):

  1. First connet: success
  2. -> terminate -> second connect: success
  3. -> terminate -> 3rd connect: Failed (not connected even after waiting)
  4. -> End of metamask -> 4th connect: Success
  5. -> terminate -> 5th connect: failure (not connected even after waiting)

Is there anything else I can do to fix the problem? If it is possible to give a guide, I will try to solve it.

abretonc7s commented 1 year ago

Hi @reese-1eco ,

Thanks for providing the details. We have an issue with current metamask wallet version where reconnection problem can occur after termination. A wallet update is incoming but may take a few more days to be available in stores.

andreahaku commented 1 year ago

@reese-1eco did the latest versions of MM mobile wallet and SDK fixed it? If so we can close the issue. Thanks for reporting.

reese-1eco commented 1 year ago

@andreahaku,

Thank you for your support. After the version upgrade, the issue situation has changed, so I will organize it and register it as a new issue.

christopherferreira9 commented 1 year ago

Thank you for reporting !