From 72d351cb3915079401fc3594baab3be50d736650 Mon Sep 17 00:00:00 2001 From: Steven Allen Date: Wed, 19 Aug 2020 10:31:52 -0700 Subject: [PATCH] fix: don't say we're sending a full wantlist unless we are (#429) I'm not sure why we set "full" to true here, but this could be the source of a whole bunch of bidirectional sync issues. That is, if two peers are syncing off each other, they could repeatedly "reset" each other's wantlist to "empty". --- internal/decision/engine.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/decision/engine.go b/internal/decision/engine.go index b620740..2a6dc60 100644 --- a/internal/decision/engine.go +++ b/internal/decision/engine.go @@ -421,7 +421,7 @@ func (e *Engine) nextEnvelope(ctx context.Context) (*Envelope, error) { } // Create a new message - msg := bsmsg.New(true) + msg := bsmsg.New(false) log.Debugw("Bitswap process tasks", "local", e.self, "taskCount", len(nextTasks)) -- GitLab