Skip to content

Send delete to all known instances#2028

Merged
BentiGorlich merged 3 commits intomainfrom
new/delete-to-all-instances
Feb 24, 2026
Merged

Send delete to all known instances#2028
BentiGorlich merged 3 commits intomainfrom
new/delete-to-all-instances

Conversation

@BentiGorlich
Copy link
Copy Markdown
Member

Previously we only send the delete activity to all instances a user interacted with. That poses a risk, because when creating content on a remote instances this content gets sent to lots of instances the user might have never interacted with. Instead of that, send the delete activity to all instances we know.

Previously we only send the delete activity to all instances a user interacted with. That poses a risk, because when creating content on a remote instances this content gets sent to lots of instances the user might have never interacted with. Instead of that, send the delete activity to all instances we know.
@BentiGorlich BentiGorlich self-assigned this Feb 22, 2026
@BentiGorlich BentiGorlich added the activitypub ActivityPub related issues label Feb 22, 2026
@blued-gear
Copy link
Copy Markdown
Collaborator

Is it enough to send it to every user, or should magazine inboxes also be included?

- also fetch magazine inboxes
- move the code to the userManager as it queries more than the user table
- fix the binding of the parameters
@BentiGorlich BentiGorlich enabled auto-merge (squash) February 24, 2026 16:10
@BentiGorlich BentiGorlich merged commit e0d4861 into main Feb 24, 2026
9 checks passed
@BentiGorlich BentiGorlich deleted the new/delete-to-all-instances branch February 24, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

activitypub ActivityPub related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants