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

I receive IDONTWANT from nimbus, even if I'm a gossipsub v1.1 node #1231

Open
ppopth opened this issue Dec 29, 2024 · 2 comments
Open

I receive IDONTWANT from nimbus, even if I'm a gossipsub v1.1 node #1231

ppopth opened this issue Dec 29, 2024 · 2 comments

Comments

@ppopth
Copy link

ppopth commented Dec 29, 2024

I ran a Prysm node which supports only Gossipsub v1.1, I expected Nimbus not to send me IDONTWANT because it has no use and waste the bandwidth, but I still receive IDONTWANT from nimbus.

@kaiserd kaiserd moved this from new to In Progress in nim-libp2p Jan 10, 2025
@kaiserd
Copy link
Collaborator

kaiserd commented Jan 10, 2025

Thanks a lot for bringing this to our attention.

Sorry for the late replay (holiday season).
I wanted to give you a quick update on the situation: In the current version of nim-libp2p, gossipsub v1.1 shouldn't be sending IDONTWANT messages anymore.
We had actually added IDONTWANT to v1.1 in a previous version as a temporary measure before v1.2 was released.
@richard-ramos is going to investigate this further and will get back with more details soon.

@richard-ramos
Copy link
Member

Looking into this, the code that fixes this issue was introduced in #113 , we're excluding sending IDONTWANT messages to peers using gossipsub < 1.2

peersToSendIDontWant.exclIfIt(
it.codec == GossipSubCodec_10 or it.codec == GossipSubCodec_11
)

I could observe that this is behaving correctly in the test case https://github.com/vacp2p/nim-libp2p/blob/master/tests/pubsub/testgossipsub.nim#L931 . This nim-libp2p version landed in nimbus stable (on v24.7.0) in status-im/nimbus-eth2#6422 on this commit: status-im/nimbus-eth2@031033a

Image

@ppopth If possible can you share more details about these peers sending you IDONTWANT messages? I wonder if those are running a very old version of nimbus

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

No branches or pull requests

3 participants