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

Handle large payloads without hitting size limit #4141

Open
124maa opened this issue Jan 18, 2024 · 5 comments
Open

Handle large payloads without hitting size limit #4141

124maa opened this issue Jan 18, 2024 · 5 comments
Assignees

Comments

@124maa
Copy link

124maa commented Jan 18, 2024

NPM Versions:

  • @walletconnect/web3wallet: ^1.10.0
  • @walletconnect/core: ^2.11.0
  • @walletconnect/sign-client: ^2.11.0

When sending messages from wallet to dApp, the payload limit is constantly reached. The limit is roughly 71kb, but the 71kb limit isn't defined anywhere. This is the error message I get when the limit is reached:

Payload size limit exceeded. Maximum size for tag 1109 is 98304

This low limit is killing the UX for dApps and is our biggest pain point with WalletConnect. It would be nice if the limit was increased, or if there were an easy way to break up a large message and send the chunks.

Our wallet is for a private blockchain, so there's unfortunately no public data provider that can reduce the load on WalletConnect.

Originally posted by @jonandgon in #4100

@ganchoradkov
Copy link
Member

hey @124maa, what size payloads are you trying to send?

@jonandgon
Copy link

Hey @ganchoradkov, I would need ~0.5 - 1.0 MB for now. As I said in the OP, we're building on a private blockchain (Aleo), and all the account data is sent from the wallet, and not through a data provider. 71kb is too little for us to properly display a dapp's state.

The ideal solution would be to have (basically) no limit, but streamed at whatever speed you decide. Or to allow wallets to run their own WalletConnect infra, but I understand that these changes would require significant work on your end.

@jonandgon
Copy link

hey @ganchoradkov any movement on increasing the payload size? Thanks!

@bkrem bkrem changed the title NPM Versions: Handle large payloads without hitting size limit Feb 8, 2024
@bkrem bkrem assigned xav Feb 9, 2024
@BrandtH22
Copy link

Hey @jonandgon , were you able to figure out a workaround or solution for this?

@jonandgon
Copy link

@BrandtH22 yes - by not using walletconnect wherever possible :)

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

No branches or pull requests

5 participants