about summary refs log tree commit diff
path: root/app/services
diff options
context:
space:
mode:
authorStarfall <us@starfall.systems>2020-12-01 10:50:41 -0600
committerStarfall <us@starfall.systems>2020-12-01 10:50:41 -0600
commit63f2eb0c84d0c51f7ece341cc3f35518a3d69520 (patch)
tree57d362f393ff7ae7fdf6b4a00d654f8dbd9c9d7c /app/services
parent19548d0a4bc303ca8017599577e21a6d522eb7bd (diff)
parent29812c2e59e02ea5ff8e4818a38b59944e2367ba (diff)
Merge branch 'glitch' into main
Diffstat (limited to 'app/services')
-rw-r--r--app/services/delete_account_service.rb15
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?