Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wallet Connect – Connection dialog don't appear after MetaMask is in background for sometime ( timeout ) - 5.6.1 #5133

Closed
jayvs6341 opened this issue Oct 20, 2022 · 11 comments
Assignees
Labels
WalletConnect WalletConnect related issue or bug

Comments

@jayvs6341
Copy link

Describe the bug

I am working on android application to connect with MetaMask using WalletConnect protocol.
The connection works fine normally. But, if the MetaMask resided in background for some time, and then we again try to connect, it don't show the dialog prompt for the connection.

I think, MetaMask has some issue with the timeout duration.

Once this happens, MetaMask don't accept connections unless we remove it from recent.

This issue happens from the very beginning versions of MetaMask, I mainly focused on the latest version ( 5.6.1 ).

Launching metamask sometimes take less than 10 sec and sometimes take more than 20 sec.

Video Link for reference of the issue

https://drive.google.com/file/d/1hN7lQUcmoozfXCLLDArAm_bWkWnvamW_/view?usp=sharing

To Reproduce

Open MetaMask through connection protocol.It cannot connect first 2 times. Need to remove MetaMask app from RECENT and connect again.

Expected behaviour

It should accept the connection even though it's in background for sometime. Because, to open MetaMask after removing it from recent takes some time to load MetaMask.

@jayvs6341
Copy link
Author

@gantunesr @andreahaku

Please help me to resolve this.

@cortisiko cortisiko added the WalletConnect WalletConnect related issue or bug label Oct 26, 2022
@andreahaku andreahaku self-assigned this Nov 8, 2022
@jayvs6341
Copy link
Author

jayvs6341 commented Nov 14, 2022

@andreahaku @gantunesr

Any update Guys ?

Very critical thing.

Hope we will resolve it very soon.

@jayvs6341
Copy link
Author

@andreahaku @gantunesr

Hello guys,

Any update ?

Hope we will get update soon.

@tommasini
Copy link
Contributor

tommasini commented Dec 12, 2022

Hey @jayvs6341, thanks for reporting.

Could you elaborate and send us the recording again (it's unavailable)?

If you connect to the MetaMask via wallet connect and the app goes to the background for a while that will not kill the connection session.

We still couldn't begin to work on this issue, thank you for your understanding!

@jayvs6341
Copy link
Author

Sure, I will share the link again.

@jayvs6341
Copy link
Author

Sure, I will share the link again.

OK, you can concentrate now.

@jayvs6341
Copy link
Author

Hey @jayvs6341, thanks for reporting.

Could you elaborate and send us the recording again (it's unavailable)?

If you connect to the MetaMask via wallet connect and the app goes to the background for a while that will not kill the connection session.

We still couldn't begin to work on this issue, thank you for your understanding!

Please find the link below.

https://drive.google.com/file/d/1sNh54SOXSkF39O_JM44iJ33wlFl_H-yh/view?usp=share_link

@tommasini
Copy link
Contributor

Thanks @jayvs6341 , could you try with the latest version of the app?

Also can you share which device are you using?

If it's possible to share the snippet of code that you are using for creating the connection, we appreciate too

@jayvs6341
Copy link
Author

Thanks @jayvs6341 , could you try with the latest version of the app?

Also can you share which device are you using?

If it's possible to share the snippet of code that you are using for creating the connection, we appreciate too

Sure, I will do this.

@jayvs6341
Copy link
Author

@tommasini

They latest build looks good.

Only issue I see is, first time it take almost 15 seconds to open the metamask.

Can we do something for it ?

@tommasini
Copy link
Contributor

tommasini commented Dec 15, 2022

Nice!
We are working towards making the opening faster, you can follow the progress on this ticket

Thanks, @jayvs6341, I will close this issue, since this last version solved this issue for you, if it happens again or you detect something else, we appreciate your support!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WalletConnect WalletConnect related issue or bug
Projects
None yet
Development

No branches or pull requests

4 participants