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

[bug] Appkit calls a lot of request to The Blockchain API on zkSync causing The 429 error with too many requests. #3294

Open
ZkSwapFinance opened this issue Nov 21, 2024 · 12 comments
Labels
bug Something isn't working needs review

Comments

@ZkSwapFinance
Copy link

Link to minimal reproducible example

https://www.zkswap.finance

Summary

Dear Reown Team,

Our dApp has been using Web3Modal (AppKit) with Next.js for a long time without any issues. However, today we encountered a problem: the "429 Too Many Requests" error.

We are using the ZKSync Era RPC, but we’re unsure why there are so many requests being sent to the Blockchain API from https://rpc.walletconnect.org/v1/?chainId=eip155%3A324&projectId=80f7c0bfe8da536240a51fe4df7277fc

The number of requests from our website has reached 7.1M, which led to WalletConnect blocking our dApp.

Could you help us understand why this is happening and how we can resolve it? Specifically, we would like to know how to disable the Blockchain API so the dApp returns to normal. Is this an issue with our integration, or is it a bug on WalletConnect's side?

Thank you for your help!

Website: https://www.zkswap.finance/swap

List of related npm package versions

Version 1.1.2

@ZkSwapFinance ZkSwapFinance added bug Something isn't working needs review labels Nov 21, 2024
@ionmig
Copy link

ionmig commented Nov 21, 2024

Hi,

we have exactly the same issue since 2 days on base chain, tried to update packages to latest versions, no success. Was working properly over the past 3 months.

also;

  • removed and reauthenticated domains on dashboard

  • tried to verify with txt file on well-known

  • used older packages to bypass

    "@reown/appkit": "^1.5.0",
    "@reown/appkit-adapter-wagmi": "^1.5.0",
    "@reown/appkit-siwe": "^1.5.0",
    "viem": "^2.21.48",
    "wagmi": "^2.13.0"

https://rpc.walletconnect.org/v1/?chainId=eip155%3A8453&projectId=XXX

note; The issue doesnt happen if you run the dapp on localhost, since localhost is always allowed under domains, i expect the issue is on the domain verification somehow.

@ZkSwapFinance
Copy link
Author

Hi,

we have exactly the same issue since 2 days on base chain, tried to update packages to latest versions, no success. Was working properly over the past 3 months.

also;

  • removed and reauthenticated domains on dashboard
  • tried to verify with txt file on well-known
  • used older packages to bypass
    "@reown/appkit": "^1.5.0",
    "@reown/appkit-adapter-wagmi": "^1.5.0",
    "@reown/appkit-siwe": "^1.5.0",
    "viem": "^2.21.48",
    "wagmi": "^2.13.0"

https://rpc.walletconnect.org/v1/?chainId=eip155%3A8453&projectId=XXX

note; The issue doesnt happen if you run the dapp on localhost, since localhost is always allowed under domains, i expect the issue is on the domain verification somehow.

Hi,

Thank you for sharing your experience—it’s reassuring to know we’re not alone in facing this issue.

Let’s wait for Reown to address this bug or determine whether the root cause lies with us or Appkit.

@lukaisailovic
Copy link
Collaborator

@ZkSwapFinance @ionmig Thanks for reporting the issue. The team is looking into it!

@rtomas
Copy link
Contributor

rtomas commented Nov 22, 2024

@ZkSwapFinance @ionmig are you using AppKit latest version ?
Do you have a site to double check it ?

https://www.zkswap.finance/ is not using AppKit.

@ionmig
Copy link

ionmig commented Nov 22, 2024

@ZkSwapFinance @ionmig are you using AppKit latest version ? Do you have a site to double check it ?

https://www.zkswap.finance/ is not using AppKit.

https://nitrodome.com

"@reown/appkit": "^1.5.0",
"@reown/appkit-adapter-wagmi": "^1.5.0",
"@reown/appkit-siwe": "^1.5.0",
"viem": "^2.21.48",
"wagmi": "^2.13.0"

@ionmig
Copy link

ionmig commented Nov 22, 2024

if you use this:
https://nitrodome.com/portal you dont need an account.
However, wait a minute and switch to dashboard and check console.

@rtomas
Copy link
Contributor

rtomas commented Nov 22, 2024

@ZkSwapFinance Can you double check if all the Allowed Origins that are in cloud are yours ?

@rtomas
Copy link
Contributor

rtomas commented Nov 22, 2024

@ZkSwapFinance are you using AppKit ?

You have 5m calls to eth_getBlockByNumber in 2days ? do you know why ?
Do you have a to reproduce example to view this calls ?

@ZkSwapFinance
Copy link
Author

@ZkSwapFinance @ionmig are you using AppKit latest version ? Do you have a site to double check it ?

https://www.zkswap.finance/ is not using AppKit.

The AppKit version is 1.1.2. At the moment, we do not have a site for testing it.

We updated another version of the wallet connect kit to ensure users are not interrupted while using the product.

@ZkSwapFinance
Copy link
Author

@ZkSwapFinance are you using AppKit ?

You have 5m calls to eth_getBlockByNumber in 2days ? do you know why ? Do you have a to reproduce example to view this calls ?

Yes, I see. We do not know why it calls so much. The old version we used, React Kit, was perfect and smooth

@ZkSwapFinance
Copy link
Author

ZkSwapFinance commented Nov 23, 2024

@ZkSwapFinance are you using AppKit ?

You have 5m calls to eth_getBlockByNumber in 2days ? do you know why ? Do you have a to reproduce example to view this calls ?

We will update the test site for you regarding this bug today.

@ZkSwapFinance
Copy link
Author

@ZkSwapFinance are you using AppKit ?

You have 5m calls to eth_getBlockByNumber in 2days ? do you know why ? Do you have a to reproduce example to view this calls ?

Here is test site: https://reown.zkswap.finance/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs review
Projects
None yet
Development

No branches or pull requests

4 participants