diff options
author | ThibG <thib@sitedethib.com> | 2020-11-21 21:27:00 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-21 21:27:00 +0100 |
commit | 66832cbf33c0800f53bc0beb5e684053259e2b36 (patch) | |
tree | c3e459e474da5adfbf2f1f7aa959d16f07eabfd0 /app | |
parent | c43f4cd3bbeaf4a6db5aa39477bb0c708422b49b (diff) |
Undo outgoing follows when suspending a remote account (#15188)
Diffstat (limited to 'app')
-rw-r--r-- | app/services/delete_account_service.rb | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/app/services/delete_account_service.rb b/app/services/delete_account_service.rb index 778d064de..9cb80c95a 100644 --- a/app/services/delete_account_service.rb +++ b/app/services/delete_account_service.rb @@ -56,6 +56,7 @@ class DeleteAccountService < BaseService @options[:skip_activitypub] = true if @options[:skip_side_effects] reject_follows! + undo_follows! purge_user! purge_profile! purge_content! @@ -79,6 +80,20 @@ class DeleteAccountService < BaseService end end + def undo_follows! + return if @account.local? || !@account.activitypub? || @options[:skip_activitypub] + + # When deleting a remote account, the account obviously doesn't + # actually become deleted on its origin server, but following relationships + # are severed on our end. Therefore, make the remote server aware that the + # follow relationships are severed to avoid confusion and potential issues + # if the remote account gets un-suspended. + + ActivityPub::DeliveryWorker.push_bulk(Follow.where(target_account: @account)) do |follow| + [Oj.dump(serialize_payload(follow, ActivityPub::UndoFollowSerializer)), follow.account_id, @account.inbox_url] + end + end + def purge_user! return if !@account.local? || @account.user.nil? |