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

Waku Relay messages detected exceeding max size of 1 MB #24

Open
3 tasks done
jm-clius opened this issue May 9, 2023 · 5 comments
Open
3 tasks done

Waku Relay messages detected exceeding max size of 1 MB #24

jm-clius opened this issue May 9, 2023 · 5 comments

Comments

@jm-clius
Copy link

jm-clius commented May 9, 2023

Background

As part of the investigations into waku-org/nwaku#1662 and #23, we've noticed that nwaku nodes are logging (and rejecting) messages on the relay (gossipsub) network of size upwards of 2 MB. The maximum size of gossip messages are 1 MB.

Problem

This issue tracks the investigation into the origin of these oversized messages.
The first step would be to verify that each of the three main Waku clients are configured to reject messages of larger than 1 MB:

A second step would be to verify that this setting does indeed work as expected practically. :)

@fryorcraken fryorcraken added this to Waku May 9, 2023
@alrevuelta
Copy link

Related vacp2p/nim-libp2p#887. The log we are seeing in nwaku is not the gossipsub message size itself but the messages encoded in the IWANT.

So this doesn't necessarily mean that someone is sending messages >1MB.

@richard-ramos
Copy link
Member

In go-libp2p-pubsub, the default max message size is set to 1 MiB https://github.com/libp2p/go-libp2p-pubsub/blob/1e161006c43f20435c2735bf24560f4d2d7eb429/pubsub.go#L249 . I did a quick test with basic2 example and can confirm that messages greater than this will be dropped.

@weboko
Copy link

weboko commented May 9, 2023

In js-waku/GossipSub I do not see any restrictions on the size of the message.
I'll to follow-up with a restriction to the size
cc: @fryorcraken

@fryorcraken
Copy link
Contributor

@weboko please create a tracking issue

@fryorcraken
Copy link
Contributor

This seems done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

5 participants