gossipsub: Preventing routing loops with timestamps and expirations#659
Draft
gossipsub: Preventing routing loops with timestamps and expirations#659
Conversation
Member
Author
|
@vyzo any opinions on this approach? Would you prefer to add an additional expiration field? |
Contributor
|
they are pretty much the same functionality wise. Expiry might be easier to work with, but timestamp is more robust i think as the decision is at the receiver. So we cant have senders set their expiry at 2262. |
Contributor
|
I also dont think we need additional field, and it can continue to function as a seqno for older clients, given sufficient resolution. We can use nanos, but micros should also be ok. |
Contributor
|
Reviewed this again today. It makes sense. I would suggest making this an extension that, when set, reinterprets the seqno as defined here. This also gives you an explicit signal for when a peer has been upgraded to this new logic. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See libp2p/go-libp2p-pubsub#573 for some context.
I'm proposing that we solve the "routing loops" issue, an issue where messages linger in the network indefinitely (or at least for long periods of time), by repurposing the
seqnofields of messages as a message timestamp. All messages would expire 2 minutes (likely configurable on a per-topic basis) after they're sent.This is a rough-draft and I'm happy to address any feedback and/or consider alternative approaches.