noisepage icon indicating copy to clipboard operation
noisepage copied to clipboard

Messenger does not handle message ID wraparound.

Open lmwnshn opened this issue 3 years ago • 0 comments

Bug Report

Summary

Messenger does not handle message ID wraparound.

Solution

If you send more than uint64_t messages, the current algorithm for tracking seen messages in docs/design_messenger.md does not work. I'm guessing that the messenger should send a signal to all its connected peers, resetting their seen message counter, before allowing wraparound to occur. But I also don't think we're going to send that many messages any time soon, so we can punt on this for a while.

lmwnshn avatar Apr 10 '21 22:04 lmwnshn