activitypub activities lack required to/cc/id fields #77

New issue

Closed

opened 2018-06-23 11:36:32 +00:00 by kaniini · 4 comments

kaniini commented

2018-06-23 11:36:32 +00:00

(Migrated from github.com)

subject covers it.

in the case of new posts being created, the objects have the to/cc fields instead of the activities themselves. this causes pure activitypub message routers like pleroma to discard the activity as invalid, since the activity lacks the to/cc fields.

also, in all cases, an activity must have an id. for example, the accept responses plume sends lack an id field.

subject covers it. in the case of new posts being created, the objects have the to/cc fields instead of the activities themselves. this causes pure activitypub message routers like pleroma to discard the activity as invalid, since the activity lacks the to/cc fields. also, in all cases, an activity must have an id. for example, the accept responses plume sends lack an id field.

kaniini commented

2018-06-23 11:43:31 +00:00

(Migrated from github.com)

other than this, Pleroma presently doesn't detect any major federation protocol violations, but i can't test any further until the to/cc are fixed, I suppose.

👍 1

elegaanz commented

2018-06-23 11:45:04 +00:00

(Migrated from github.com)

Does Pleroma support the Article type? (and thanks for the reports btw)

Does Pleroma support the `Article` type? (and thanks for the reports btw)

kaniini commented

2018-06-23 11:46:32 +00:00

(Migrated from github.com)

my testing instance does have awareness of Article items, but it's not enabled in mainline quite yet because nobody was using it. we treat them the same as notes, since pleroma will autocollapse any data larger than 1KB.

my testing instance does have awareness of `Article` items, but it's not enabled in mainline quite yet because nobody was using it. we treat them the same as notes, since pleroma will autocollapse any data larger than 1KB.

👍 1

elegaanz commented

2018-06-23 12:51:39 +00:00

(Migrated from github.com)

All activities should have these fields now. And the demo instance has been updated if you want to check.

🎉 1