Try to handle misconfiguration scenarios gracefully

This commit is contained in:
Mark Felder 2020-10-08 12:09:31 -05:00 committed by Mark Felder
parent 8b81d62227
commit fba770b3ea

View file

@ -1,34 +1,49 @@
defmodule Pleroma.Web.ActivityPub.MRF.FollowBotPolicy do defmodule Pleroma.Web.ActivityPub.MRF.FollowBotPolicy do
@behaviour Pleroma.Web.ActivityPub.MRF @behaviour Pleroma.Web.ActivityPub.MRF
alias Pleroma.Config
alias Pleroma.User alias Pleroma.User
alias Pleroma.Web.CommonAPI alias Pleroma.Web.CommonAPI
require Logger require Logger
@impl true @impl true
def filter(message) do def filter(message) do
with follower_nickname <- Config.get([:mrf_follow_bot, :follower_nickname]),
%User{} = follower <- User.get_cached_by_nickname(follower_nickname),
%{"type" => "Create", "object" => %{"type" => "Note"}} <- message do
try_follow(follower, message)
else
nil ->
Logger.warn(
"#{__MODULE__} skipped because of missing :mrf_follow_bot, :follower_nickname configuration or the account
does not exist."
)
{:ok, message}
_ ->
{:ok, message}
end
end
defp try_follow(follower, message) do
Task.start(fn -> Task.start(fn ->
follower_nickname = Pleroma.Config.get([:mrf_follow_bot, :follower_nickname]) to = Map.get(message, "to", [])
cc = Map.get(message, "cc", [])
actor = [message["actor"]]
with %User{} = follower <- User.get_cached_by_nickname(follower_nickname), Enum.concat([to, cc, actor])
%{"type" => "Create", "object" => %{"type" => "Note"}} <- message do |> List.flatten()
to = Map.get(message, "to", []) |> User.get_all_by_ap_id()
cc = Map.get(message, "cc", []) |> Enum.each(fn user ->
actor = [message["actor"]] Logger.info("Checking if #{user.nickname} can be followed")
Enum.concat([to, cc, actor]) with false <- User.following?(follower, user),
|> List.flatten() false <- user.locked,
|> User.get_all_by_ap_id() false <- (user.bio || "") |> String.downcase() |> String.contains?("nobot") do
|> Enum.each(fn user -> Logger.info("Following #{user.nickname}")
Logger.info("Checking if #{user.nickname} can be followed") CommonAPI.follow(follower, user)
end
with false <- User.following?(follower, user), end)
false <- user.locked,
false <- (user.bio || "") |> String.downcase() |> String.contains?("nobot") do
Logger.info("Following #{user.nickname}")
CommonAPI.follow(follower, user)
end
end)
end
end) end)
{:ok, message} {:ok, message}