libp2p / libp2p/go-libp2p-pubsub

Broadcast Loops

Open
#573 17 comments 0 reactions 1 assignee View on GitHub

@Stebalien is already working on this.

Since Aug 19, 2024.

Dominant language
Go
Stars
364
Forks
221
PR merge metrics
No merged PRs in 30d

Description

In F3, we've run into some pretty severe broadcast loops. From what we can tell, the time cache is _basically_ useless.

The core issue is that we have _NO_ TTLs and/or expiration on messages. Personally, I'd recommend a simple expiration to all messages that's strictly less than the time cache duration. We can even treat the time cache duration as a network parameter, refusing to propagate messages that have an expiration more than that duration in the future.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.