Merge branch 'fix/hellthread-policy-recipient-check' into 'develop'

Fix recipient count in hellthread policy

See merge request pleroma/pleroma!836

lib/pleroma/web/activity_pub/mrf/hellthread_policy.ex

@@ -12,14 +12,14 @@ defp delist_message(message, threshold) when threshold > 0 do
     follower_collection? = Enum.member?(message["to"] ++ message["cc"], follower_collection)
 
     message =
-      case recipients = get_recipient_count(message) do
+      case get_recipient_count(message) do
-        {:public, _}
+        {:public, recipients}
         when follower_collection? and recipients > threshold ->
           message
           |> Map.put("to", [follower_collection])
           |> Map.put("cc", ["https://www.w3.org/ns/activitystreams#Public"])
 
-        {:public, _} when recipients > threshold ->
+        {:public, recipients} when recipients > threshold ->
           message
           |> Map.put("to", [])
           |> Map.put("cc", ["https://www.w3.org/ns/activitystreams#Public"])

test/web/activity_pub/mrf/hellthread_policy_test.exs

@@ -8,7 +8,6 @@ defmodule Pleroma.Web.ActivityPub.MRF.HellthreadPolicyTest do
 
   import Pleroma.Web.ActivityPub.MRF.HellthreadPolicy
 
-  describe "hellthread filter tests" do
   setup do
     user = insert(:user)
 
@@ -18,22 +17,38 @@ defmodule Pleroma.Web.ActivityPub.MRF.HellthreadPolicyTest do
       "type" => "Create",
       "to" => [
         "https://www.w3.org/ns/activitystreams#Public",
-          "https://instace.tld/users/user1",
+        "https://instance.tld/users/user1",
-          "https://instace.tld/users/user2",
+        "https://instance.tld/users/user2",
-          "https://instace.tld/users/user3"
+        "https://instance.tld/users/user3"
       ]
     }
 
     [user: user, message: message]
   end
 
-    test "reject test", %{message: message} do
+  describe "reject" do
+    test "rejects the message if the recipient count is above reject_threshold", %{
+      message: message
+    } do
       Pleroma.Config.put([:mrf_hellthread], %{delist_threshold: 0, reject_threshold: 2})
 
       {:reject, nil} = filter(message)
     end
 
-    test "delist test", %{user: user, message: message} do
+    test "does not reject the message if the recipient count is below reject_threshold", %{
+      message: message
+    } do
+      Pleroma.Config.put([:mrf_hellthread], %{delist_threshold: 0, reject_threshold: 3})
+
+      assert {:ok, ^message} = filter(message)
+    end
+  end
+
+  describe "delist" do
+    test "delists the message if the recipient count is above delist_threshold", %{
+      user: user,
+      message: message
+    } do
       Pleroma.Config.put([:mrf_hellthread], %{delist_threshold: 2, reject_threshold: 0})
 
       {:ok, message} = filter(message)
@@ -41,10 +56,18 @@ test "delist test", %{user: user, message: message} do
       assert "https://www.w3.org/ns/activitystreams#Public" in message["cc"]
     end
 
+    test "does not delist the message if the recipient count is below delist_threshold", %{
+      message: message
+    } do
+      Pleroma.Config.put([:mrf_hellthread], %{delist_threshold: 4, reject_threshold: 0})
+
+      assert {:ok, ^message} = filter(message)
+    end
+  end
+
   test "excludes follower collection and public URI from threshold count", %{message: message} do
     Pleroma.Config.put([:mrf_hellthread], %{delist_threshold: 0, reject_threshold: 3})
 
-      {:ok, _} = filter(message)
+    assert {:ok, ^message} = filter(message)
-    end
   end
 end