mirrors/gotosocial
1
0
Fork 0
mirror of https://github.com/superseriousbusiness/gotosocial.git synced 2025-10-30 18:42:26 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

[bugfix] Fix replies not being stored pending approval (#3409)

Browse source
This commit is contained in:
tobi 2024-10-09 11:02:10 +02:00 • committed by GitHub
commit e8fd40f3ca
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 39 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

61
internal/federation/dereferencing/status_permitted.go
View file

@ -235,7 +235,7 @@ func (d *Dereferencer) isPermittedReply(
// Status doesn't claim to be approved. // Status doesn't claim to be approved.
// Check interaction policy of inReplyTo // Check interaction policy of inReplyTo
// to see if it doesn't require approval. // to see what we need to do with it.
replyable, err := d.intFilter.StatusReplyable(ctx, replyable, err := d.intFilter.StatusReplyable(ctx,
reply.Account, reply.Account,
inReplyTo, inReplyTo,
@ -260,35 +260,52 @@ func (d *Dereferencer) isPermittedReply(
) )
} }
// Reply is permitted according to the interaction if replyable.Permitted() &&
// policy set on the replied-to status (if any). !replyable.MatchedOnCollection() {
// Reply is permitted and match was *not* made
if !replyable.MatchedOnCollection() { // based on inclusion in a followers/following
// If we didn't match on a collection, // collection. Just permit the reply full stop
// then we don't require an acceptIRI, // as no approval / accept URI is necessary.
// and we don't need to send an Accept;
// just permit the reply full stop.
return true, nil return true, nil
} }
// Reply is permitted, but match was made based // Reply is either permitted based on inclusion in a
// on inclusion in a followers/following collection. // followers/following collection, *or* is permitted
// // pending approval, though we know at this point
// If the status is ours, mark it as PreApproved // that the status did not include an approvedBy URI.
// so the processor knows to create and send out
// an Accept for it immediately. if !inReplyTo.IsLocal() {
if inReplyTo.IsLocal() { // If the replied-to status is remote, we should just
// drop this reply at this point, as we can't verify
// that the remote replied-to account approves it, and
// we can't verify the presence of a remote account
// in one of another remote account's collections.
//
// It's possible we'll get an Accept from the replied-
// to account later, and we can store this reply then.
return false, nil
}
// Replied-to status is ours, so the
// replied-to account is ours as well.
if replyable.MatchedOnCollection() {
// If permission was granted based on inclusion in
// a followers/following collection, pre-approve the
// reply, as we ourselves can validate presence of the
// replier in the appropriate collection. Pre-approval
// lets the processor know it should send out an Accept
// straight away on behalf of the replied-to account.
reply.PendingApproval = util.Ptr(true) reply.PendingApproval = util.Ptr(true)
reply.PreApproved = true reply.PreApproved = true
return true, nil return true, nil
} }
// For replies to remote statuses, which matched // Reply just requires approval from the local account
// on a followers/following collection, but did not // it replies to. Set PendingApproval so the processor
// include an acceptIRI, we should just drop it. // knows to create a pending interaction request.
// It's possible we'll get an Accept for it later reply.PendingApproval = util.Ptr(true)
// and we can check everything again. return true, nil
return false, nil
} }
// unpermittedByParent marks the given reply as rejected // unpermittedByParent marks the given reply as rejected