Browse Source

Resolve unknown status from Add activity, skip Remove if unknown (#7526)

Fix #7518
Eugen Rochko 5 years ago
parent
commit
57b503d4ef

+ 9 - 0
app/lib/activitypub/activity.rb

@@ -118,4 +118,13 @@ class ActivityPub::Activity
   def delete_later!(uri)
   def delete_later!(uri)
     redis.setex("delete_upon_arrival:#{@account.id}:#{uri}", 6.hours.seconds, uri)
     redis.setex("delete_upon_arrival:#{@account.id}:#{uri}", 6.hours.seconds, uri)
   end
   end
+
+  def fetch_remote_original_status
+    if object_uri.start_with?('http')
+      return if ActivityPub::TagManager.instance.local_uri?(object_uri)
+      ActivityPub::FetchRemoteStatusService.new.call(object_uri, id: true, on_behalf_of: @account.followers.local.first)
+    elsif @object['url'].present?
+      ::FetchRemoteStatusService.new.call(@object['url'])
+    end
+  end
 end
 end

+ 3 - 2
app/lib/activitypub/activity/add.rb

@@ -4,9 +4,10 @@ class ActivityPub::Activity::Add < ActivityPub::Activity
   def perform
   def perform
     return unless @json['target'].present? && value_or_id(@json['target']) == @account.featured_collection_url
     return unless @json['target'].present? && value_or_id(@json['target']) == @account.featured_collection_url
 
 
-    status = status_from_uri(object_uri)
+    status   = status_from_uri(object_uri)
+    status ||= fetch_remote_original_status
 
 
-    return unless status.account_id == @account.id && !@account.pinned?(status)
+    return unless !status.nil? && status.account_id == @account.id && !@account.pinned?(status)
 
 
     StatusPin.create!(account: @account, status: status)
     StatusPin.create!(account: @account, status: status)
   end
   end

+ 0 - 10
app/lib/activitypub/activity/announce.rb

@@ -26,16 +26,6 @@ class ActivityPub::Activity::Announce < ActivityPub::Activity
 
 
   private
   private
 
 
-  def fetch_remote_original_status
-    if object_uri.start_with?('http')
-      return if ActivityPub::TagManager.instance.local_uri?(object_uri)
-
-      ActivityPub::FetchRemoteStatusService.new.call(object_uri, id: true, on_behalf_of: @account.followers.local.first)
-    elsif @object['url'].present?
-      ::FetchRemoteStatusService.new.call(@object['url'])
-    end
-  end
-
   def announceable?(status)
   def announceable?(status)
     status.account_id == @account.id || status.public_visibility? || status.unlisted_visibility?
     status.account_id == @account.id || status.public_visibility? || status.unlisted_visibility?
   end
   end

+ 1 - 1
app/lib/activitypub/activity/remove.rb

@@ -6,7 +6,7 @@ class ActivityPub::Activity::Remove < ActivityPub::Activity
 
 
     status = status_from_uri(object_uri)
     status = status_from_uri(object_uri)
 
 
-    return unless status.account_id == @account.id
+    return unless !status.nil? && status.account_id == @account.id
 
 
     pin = StatusPin.find_by(account: @account, status: status)
     pin = StatusPin.find_by(account: @account, status: status)
     pin&.destroy!
     pin&.destroy!

+ 22 - 3
spec/lib/activitypub/activity/add_spec.rb

@@ -18,12 +18,31 @@ RSpec.describe ActivityPub::Activity::Add do
   describe '#perform' do
   describe '#perform' do
     subject { described_class.new(json, sender) }
     subject { described_class.new(json, sender) }
 
 
-    before do
+    it 'creates a pin' do
       subject.perform
       subject.perform
+      expect(sender.pinned?(status)).to be true
     end
     end
 
 
-    it 'creates a pin' do
-      expect(sender.pinned?(status)).to be true
+    context 'when status was not known before' do
+      let(:json) do
+        {
+          '@context': 'https://www.w3.org/ns/activitystreams',
+          id: 'foo',
+          type: 'Add',
+          actor: ActivityPub::TagManager.instance.uri_for(sender),
+          object: 'https://example.com/unknown',
+          target: sender.featured_collection_url,
+        }.with_indifferent_access
+      end
+
+      before do
+        stub_request(:get, 'https://example.com/unknown').to_return(status: 410)
+      end
+
+      it 'fetches the status' do
+        subject.perform
+        expect(a_request(:get, 'https://example.com/unknown')).to have_been_made.at_least_once
+      end
     end
     end
   end
   end
 end
 end