about summary refs log tree commit diff
path: root/app/services
diff options
context:
space:
mode:
authorJeong Arm <kjwonmail@gmail.com>2022-08-16 03:32:21 +0900
committerGitHub <noreply@github.com>2022-08-15 20:32:21 +0200
commit6aa83b13ba541b948d7d6927728bbdcf63a23950 (patch)
treee5efbe6df450a5c88e72db330b330b7e79ff75cd /app/services
parent113dd90aa3e8dd930fa343a2dea52e64bf63eb06 (diff)
Properly delete remote account's avatar/header when fetch/update (#18973)
Diffstat (limited to 'app/services')
-rw-r--r--app/services/activitypub/process_account_service.rb2
1 files changed, 2 insertions, 0 deletions
diff --git a/app/services/activitypub/process_account_service.rb b/app/services/activitypub/process_account_service.rb
index 4449a5427..34750dba6 100644
--- a/app/services/activitypub/process_account_service.rb
+++ b/app/services/activitypub/process_account_service.rb
@@ -105,11 +105,13 @@ class ActivityPub::ProcessAccountService < BaseService
   def set_fetchable_attributes!
     begin
       @account.avatar_remote_url = image_url('icon') || '' unless skip_download?
+      @account.avatar = nil if @account.avatar_remote_url.blank?
     rescue Mastodon::UnexpectedResponseError, HTTP::TimeoutError, HTTP::ConnectionError, OpenSSL::SSL::SSLError
       RedownloadAvatarWorker.perform_in(rand(30..600).seconds, @account.id)
     end
     begin
       @account.header_remote_url = image_url('image') || '' unless skip_download?
+      @account.header = nil if @account.header_remote_url.blank?
     rescue Mastodon::UnexpectedResponseError, HTTP::TimeoutError, HTTP::ConnectionError, OpenSSL::SSL::SSLError
       RedownloadHeaderWorker.perform_in(rand(30..600).seconds, @account.id)
     end