From 2c2dbe318e2d01f4d7dbbade3970684e4f0b9f6b Mon Sep 17 00:00:00 2001
From: Tobi Smethurst <31960611+tsmethurst@users.noreply.github.com>
Date: Tue, 27 Jul 2021 10:45:22 +0200
Subject: federating db updates (#118)

---
 internal/federation/federatingdb/undo.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'internal/federation/federatingdb/undo.go')

diff --git a/internal/federation/federatingdb/undo.go b/internal/federation/federatingdb/undo.go
index 063adaedd..dd82e7bac 100644
--- a/internal/federation/federatingdb/undo.go
+++ b/internal/federation/federatingdb/undo.go
@@ -33,7 +33,9 @@ func (f *federatingDB) Undo(ctx context.Context, undo vocab.ActivityStreamsUndo)
 
 	targetAcctI := ctx.Value(util.APAccount)
 	if targetAcctI == nil {
-		l.Error("UNDO: target account wasn't set on context")
+		// If the target account wasn't set on the context, that means this request didn't pass through the
+		// API, but came from inside GtS as the result of another activity on this instance. That being so,
+		// we can safely just ignore this activity, since we know we've already processed it elsewhere.
 		return nil
 	}
 	targetAcct, ok := targetAcctI.(*gtsmodel.Account)
-- 
cgit v1.2.3