-
Notifications
You must be signed in to change notification settings - Fork 45
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
feat: skip strict checking for discovering peerExchange peers #1136
Comments
Looks like it may not be an ENR related issue, but noticing these logs in status-desktop. This should not happen as the peer-exchange peers are added with pubsubTopics in peerstore. Need to dig deeper to understand what is going on.
|
As peerExchange is not defined in ENR and it is not registered as waku protocol hence it is not included in serviceSlots which is causing these logs to happen during on-demand discovery. Hence, these logs can be ignored for now as they don't mean anything going wrong. |
Problem
On-Demand discovery is not able to select peers for peerExchange, since that is not registered as a waku protocol and also the ENR for peers don't have this information.
So, this check can be laxed and we can assume that any serviceNode in the network will support peerExchange and use them for peerExchange. Worst case there would be failure identified during stream negotiation which is still fine.
More details here: https://discord.com/channels/1110799176264056863/1242773830016176129
Alternatives considered
Update Waku ENR to include peerExchange bit capability. This is more complex and require further research hence not taking this path for now.
Additional context
Update status-go so that light clients discovery can stop once some limit in peer store is reached. Some housekeeping could be run from time to time, but this could be much less often than every 5 seconds.
The text was updated successfully, but these errors were encountered: