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

Safe connector stucks reconnecting state if phantom Wallet installed as extension #4236

Closed
1 task done
yulafezmesi opened this issue Sep 2, 2024 · 2 comments
Closed
1 task done

Comments

@yulafezmesi
Copy link
Contributor

yulafezmesi commented Sep 2, 2024

Check existing issues

Describe the bug

i'm try to connect Safe connector however it's stucks reconnecting state while Phantom Wallet installed as browser extension. when we disable Phantom Wallet, it will be autoconnect without an issues.

Link to Minimal Reproducible Example

https://app.safe.global/apps/open?safe=base:0x03e030BE8Ad2A37aE849D2710d3d567F7005d2FA&appUrl=https://9114b809.new-wagmi-wtopom.pages.dev/

Steps To Reproduce

open this safe app URL while Phantom Wallet installed, you'll noticed the address that you connected however the connection state is hanging reconnecting

image

source
https://github.com/yulafezmesi/new-wagmi-wtopom

What Wagmi package(s) are you using?

wagmi

Wagmi Package(s) Version(s)

2.12.8

Viem Version

2.21.1

TypeScript Version

5.2.2

Anything else?

@yulafezmesi
Copy link
Contributor Author

i've found the issue and fixed here, not sure perfect workaround please let me know if you have any recommendation.

Thanks.

@tmm
Copy link
Member

tmm commented Sep 16, 2024

This is considered a wallet issue as it can cause unintended side effects. Best to follow up with Phantom so they can fix their implementation of 'eth_accounts'.

@tmm tmm closed this as not planned Won't fix, can't repro, duplicate, stale Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants