You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If some peers send ConsensusMessage for different round too often, recipient will add those peers to deny list, and will deny messages from that peer till new height has been started.
This logic prevents VoteSet from keeping too many messages.
But there are no protection logics for different height ConsensusMessages. (This will affect both of Gossip memory and VoteSet)
Also, there are no protection logics for invalid ConsensusMessages. (This will affect Gossip memory)
We can assume those spam filter logic is up to network or transport layer, and do nothing. (Assume Libplanet is not responsible)
But I think it's beneficial for Libplanet to have protection logics on the view of product, if we can do something for that.
TO-BE
Spam filter logic to cover different height messages, and invalid messages
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
AS-IS
ConsensusMessage
for different round too often, recipient will add those peers to deny list, and will deny messages from that peer till new height has been started.VoteSet
from keeping too many messages.ConsensusMessage
s. (This will affect both of Gossip memory and VoteSet)ConsensusMessage
s. (This will affect Gossip memory)TO-BE
Want to discuss
Related issue
GossipConsensusMessageCommunicator
#3266Beta Was this translation helpful? Give feedback.
All reactions