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: peers are descored due to message validation failure #4660

Closed
chaitanyaprem opened this issue Feb 1, 2024 · 3 comments
Closed

bug: peers are descored due to message validation failure #4660

chaitanyaprem opened this issue Feb 1, 2024 · 3 comments
Assignees
Labels
status-waku-integ All issues relating to the Status Waku integration.

Comments

@chaitanyaprem
Copy link
Contributor

Problem

It has been noticed a couple of times by @mprakhov and @anastasiyaig that relay is not receiving some of all messages.
On analysis it has been observed that peer scores for all peers have been dropped too low causing some of them to move to graylist and ignornig IWANT/IHAVE messages from others.

Root Cause seems to be message validation failure which occurs due to unable to decrypt messages received on particular shard/pubsubTopic.

Ref discussion and analysis #4459 (comment)

Per @richard-ramos

I believe this problem might be happening due to https://github.com/status-im/status-desktop/issues/13267 . I suspect that a shard assignment removal or change happened before, but the private key used originally for the community was not removed, so the topic validation in relay would still be applying a wrong key, and end up penalizing peers by mistake.

if the problem is due to that, then likely it is fixed in https://github.com/status-im/status-go/pull/4627 (this is already in desktop master)

@anastasiyaig Do update once you test the nightly build in your windows machine where you saw this issue recently and update.

@chaitanyaprem chaitanyaprem added the status-waku-integ All issues relating to the Status Waku integration. label Feb 1, 2024
@chaitanyaprem chaitanyaprem self-assigned this Feb 1, 2024
@chaitanyaprem
Copy link
Contributor Author

chaitanyaprem commented Feb 12, 2024

Workaround is to use different shard for each community that is being created .
In order to avoid conflict amongst users, maybe a common page to be maintained where users indicate which shard is being used for each community in order to avoid such issues.

We must not reuse same shard for 2 communities as this will lead to validation failure. More information mentioned in waku-org/go-waku#1022

@chaitanyaprem chaitanyaprem removed their assignment Feb 21, 2024
@chaitanyaprem
Copy link
Contributor Author

Related to status-im/status-desktop#1

Looks like wrong reference @chair28980 .

@chair28980
Copy link

Closing per discussion with Status QA, issue not observed in latest release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status-waku-integ All issues relating to the Status Waku integration.
Projects
None yet
Development

No branches or pull requests

3 participants