Open ufarooqstatus opened 7 months ago
Attention: 2 lines
in your changes are missing coverage. Please review.
Comparison is base (
fe4ff79
) 83.28% compared to head (3fb83b8
) 83.31%.
this is a good idea - in fact, it should be extended so that it also skips messages that were sent by the other node "in full" while queued.
Basically, a peer can tell us they have the message already in two ways: by IDONTWANT and by sending the message in its entirety.
The latter is quite possible: because low-prio messages are the ones being forwarded across the network in bursts, it's possible we receive (a duplicate of) it from the peer while sending some other message.
Remove peers from the relay (ToSendPeers) list, from which message or idontwant is received. We delay this action till sendMsg to increase such peer count.