Allow pin data fetching to fail

lib/pleroma/web/activity_pub/activity_pub.ex

@@ -1792,6 +1792,11 @@ def pin_data_from_featured_collection(
     end)
   end
 
+  def pin_data_from_featured_collection(obj) do
+    Logger.error("Could not parse featured collection #{inspect(obj)}")
+    %{}
+  end
+
   def fetch_and_prepare_featured_from_ap_id(nil) do
     {:ok, %{}}
   end

test/pleroma/web/activity_pub/activity_pub_test.exs

@@ -2629,6 +2629,14 @@ test "allow fetching of accounts with an empty string name field" do
     assert user.name == " "
   end
 
+  test "pin_data_from_featured_collection will ignore unsupported values" do
+    assert %{} ==
+             ActivityPub.pin_data_from_featured_collection(%{
+               "type" => "CollectionThatIsNotRealAndCannotHurtMe",
+               "first" => "https://social.example/users/alice/collections/featured?page=true"
+             })
+  end
+
   describe "persist/1" do
     test "should not persist remote delete activities" do
       poster = insert(:user, local: false)