summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Gleason <alex@alexgleason.me>2020-07-29 12:58:08 -0500
committerAlex Gleason <alex@alexgleason.me>2020-07-29 12:58:08 -0500
commit93638935d783c092dabac51982426ebd98a21e0e (patch)
treedf56a73d603f36cc0b3725c3ed6ebe20a3fa410c
parent2a99e7df8e3c5c5c6cdf15bff56d0258c9a5287e (diff)
SimpleMRF: :silence --> :followers_only
-rw-r--r--config/description.exs2
-rw-r--r--docs/configuration/cheatsheet.md2
-rw-r--r--lib/pleroma/web/activity_pub/mrf/simple_policy.ex10
-rw-r--r--test/web/activity_pub/mrf/simple_policy_test.exs8
4 files changed, 11 insertions, 11 deletions
diff --git a/config/description.exs b/config/description.exs
index 9ffe6f93d..dc6e2a76e 100644
--- a/config/description.exs
+++ b/config/description.exs
@@ -1525,7 +1525,7 @@ config :pleroma, :config_description, [
suggestions: ["example.com", "*.example.com"]
},
%{
- key: :silence,
+ key: :followers_only,
type: {:list, :string},
description: "Force posts from the given instances to be visible by followers only",
suggestions: ["example.com", "*.example.com"]
diff --git a/docs/configuration/cheatsheet.md b/docs/configuration/cheatsheet.md
index 9a7f4f369..b195b6f17 100644
--- a/docs/configuration/cheatsheet.md
+++ b/docs/configuration/cheatsheet.md
@@ -122,7 +122,7 @@ To add configuration to your config file, you can copy it from the base config.
* `federated_timeline_removal`: List of instances to remove from Federated (aka The Whole Known Network) Timeline.
* `reject`: List of instances to reject any activities from.
* `accept`: List of instances to accept any activities from.
-* `silence`: List of instances to force posts as followers-only.
+* `followers_only`: List of instances to force posts as followers-only.
* `report_removal`: List of instances to reject reports from.
* `avatar_removal`: List of instances to strip avatars from.
* `banner_removal`: List of instances to strip banners from.
diff --git a/lib/pleroma/web/activity_pub/mrf/simple_policy.ex b/lib/pleroma/web/activity_pub/mrf/simple_policy.ex
index 4dce22cfa..ffaac767e 100644
--- a/lib/pleroma/web/activity_pub/mrf/simple_policy.ex
+++ b/lib/pleroma/web/activity_pub/mrf/simple_policy.ex
@@ -109,13 +109,13 @@ defmodule Pleroma.Web.ActivityPub.MRF.SimplePolicy do
{:ok, object}
end
- defp check_silence(%{host: actor_host} = _actor_info, object) do
- silence =
- Config.get([:mrf_simple, :silence])
+ defp check_followers_only(%{host: actor_host} = _actor_info, object) do
+ followers_only =
+ Config.get([:mrf_simple, :followers_only])
|> MRF.subdomains_regex()
object =
- with true <- MRF.subdomain_match?(silence, actor_host),
+ with true <- MRF.subdomain_match?(followers_only, actor_host),
user <- User.get_cached_by_ap_id(object["actor"]) do
# Don't use Map.get/3 intentionally, these must not be nil
fixed_to = object["to"] || []
@@ -200,7 +200,7 @@ defmodule Pleroma.Web.ActivityPub.MRF.SimplePolicy do
{:ok, object} <- check_media_removal(actor_info, object),
{:ok, object} <- check_media_nsfw(actor_info, object),
{:ok, object} <- check_ftl_removal(actor_info, object),
- {:ok, object} <- check_silence(actor_info, object),
+ {:ok, object} <- check_followers_only(actor_info, object),
{:ok, object} <- check_report_removal(actor_info, object) do
{:ok, object}
else
diff --git a/test/web/activity_pub/mrf/simple_policy_test.exs b/test/web/activity_pub/mrf/simple_policy_test.exs
index 510a31d80..c0e82731b 100644
--- a/test/web/activity_pub/mrf/simple_policy_test.exs
+++ b/test/web/activity_pub/mrf/simple_policy_test.exs
@@ -16,7 +16,7 @@ defmodule Pleroma.Web.ActivityPub.MRF.SimplePolicyTest do
federated_timeline_removal: [],
report_removal: [],
reject: [],
- silence: [],
+ followers_only: [],
accept: [],
avatar_removal: [],
banner_removal: [],
@@ -263,9 +263,9 @@ defmodule Pleroma.Web.ActivityPub.MRF.SimplePolicyTest do
end
end
- describe "when :silence" do
+ describe "when :followers_only" do
test "is empty" do
- Config.put([:mrf_simple, :silence], [])
+ Config.put([:mrf_simple, :followers_only], [])
{_, ftl_message} = build_ftl_actor_and_message()
local_message = build_local_message()
@@ -296,7 +296,7 @@ defmodule Pleroma.Web.ActivityPub.MRF.SimplePolicyTest do
|> URI.parse()
|> Map.fetch!(:host)
- Config.put([:mrf_simple, :silence], [actor_domain])
+ Config.put([:mrf_simple, :followers_only], [actor_domain])
assert {:ok, new_activity} = SimplePolicy.filter(activity)
assert actor.follower_address in new_activity["to"]