We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
PeerManager.prunePeer should not use ENR as it's easy to fake.
PeerManager.prunePeer
ENR
Low
https://github.com/waku-org/nwaku/blob/master/waku/node/peer_manager/peer_manager.nim#L852-L853
Instead of ENR, GossipSub.peerStats could be an option.
GossipSub.peerStats
wakunode2: v0.24.0-rc.0-126-g4117fe branch: test-peer-connection-management commit: bbebbac (might change in the future)
CC: @alrevuelta
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Problem
PeerManager.prunePeer
should not useENR
as it's easy to fake.Impact
Low
To reproduce
https://github.com/waku-org/nwaku/blob/master/waku/node/peer_manager/peer_manager.nim#L852-L853
Expected behavior
Instead of
ENR
,GossipSub.peerStats
could be an option.nwaku version/commit hash
wakunode2: v0.24.0-rc.0-126-g4117fe
branch: test-peer-connection-management
commit: bbebbac (might change in the future)
CC: @alrevuelta
The text was updated successfully, but these errors were encountered: