qugalet/gotosocial
1
0
Fork 0
mirror of https://github.com/superseriousbusiness/gotosocial.git synced 2025-02-04 20:05:05 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[bugfix] Don't panic on delivery of Activity with no object (#3730)

Browse source
This commit is contained in:
tobi 2025-02-03 12:52:24 +01:00 committed by GitHub
parent acd3e80ae1
commit 053d820845
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 27 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
internal/federation/federatingprotocol.go
View file

@ -116,8 +116,10 @@ func (f *Federator) PostInboxRequestBodyHook(ctx context.Context, r *http.Reques
otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...) otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...)
} }
// Now perform the same checks, but for the Object(s) of the Activity. // Now perform the same checks, but
// for any Object(s) of the Activity.
objectProp := activity.GetActivityStreamsObject() objectProp := activity.GetActivityStreamsObject()
if objectProp != nil {
for iter := objectProp.Begin(); iter != objectProp.End(); iter = iter.Next() { for iter := objectProp.Begin(); iter != objectProp.End(); iter = iter.Next() {
if iter.IsIRI() { if iter.IsIRI() {
otherIRIs = append(otherIRIs, iter.GetIRI()) otherIRIs = append(otherIRIs, iter.GetIRI())
@ -145,6 +147,7 @@ func (f *Federator) PostInboxRequestBodyHook(ctx context.Context, r *http.Reques
otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...) otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...)
} }
} }
}
// Clean any instances of the public URI, since // Clean any instances of the public URI, since
// we don't care about that in this context. // we don't care about that in this context.