summaryrefslogtreecommitdiff
path: root/internal/federation/federatingdb/undo.go
diff options
context:
space:
mode:
authorLibravatar Tobi Smethurst <31960611+tsmethurst@users.noreply.github.com>2021-07-27 10:45:22 +0200
committerLibravatar GitHub <noreply@github.com>2021-07-27 10:45:22 +0200
commit2c2dbe318e2d01f4d7dbbade3970684e4f0b9f6b (patch)
tree0565cb6e2ce8a3f7f96fb2260fbdc0b7511abd87 /internal/federation/federatingdb/undo.go
parentdoc updates (#117) (diff)
downloadgotosocial-2c2dbe318e2d01f4d7dbbade3970684e4f0b9f6b.tar.xz
federating db updates (#118)
Diffstat (limited to 'internal/federation/federatingdb/undo.go')
-rw-r--r--internal/federation/federatingdb/undo.go4
1 files changed, 3 insertions, 1 deletions
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)