Do not rely on activity arriving exactly once after delete arrived (#4754)

master
Eugen Rochko 7 years ago committed by GitHub
parent a75aa62f5b
commit c2980d5b17
  1. 9
      app/lib/activitypub/activity.rb
  2. 6
      app/lib/activitypub/activity/delete.rb

@ -97,14 +97,7 @@ class ActivityPub::Activity
end end
def delete_arrived_first?(uri) def delete_arrived_first?(uri)
key = "delete_upon_arrival:#{@account.id}:#{uri}" redis.exists("delete_upon_arrival:#{@account.id}:#{uri}")
if redis.exists(key)
redis.del(key)
true
else
false
end
end end
def delete_later!(uri) def delete_later!(uri)

@ -5,13 +5,13 @@ class ActivityPub::Activity::Delete < ActivityPub::Activity
status = Status.find_by(uri: object_uri, account: @account) status = Status.find_by(uri: object_uri, account: @account)
status ||= Status.find_by(uri: @object['_:atomUri'], account: @account) if @object.is_a?(Hash) && @object['_:atomUri'].present? status ||= Status.find_by(uri: @object['_:atomUri'], account: @account) if @object.is_a?(Hash) && @object['_:atomUri'].present?
if status.nil?
delete_later!(object_uri) delete_later!(object_uri)
else
return if status.nil?
forward_for_reblogs(status) forward_for_reblogs(status)
delete_now!(status) delete_now!(status)
end end
end
private private

Loading…
Cancel
Save