Merge branch 'bugfix/broken-thread-filtering-false-positive' into 'develop'

activitypub: fix possible false positives with broken thread filtering

See merge request pleroma/pleroma!398
This commit is contained in:
kaniini 2018-10-28 05:52:51 +00:00
commit 39103a91be

View File

@ -793,9 +793,10 @@ defmodule Pleroma.Web.ActivityPub.ActivityPub do
# child # child
def entire_thread_visible_for_user?( def entire_thread_visible_for_user?(
%Activity{data: %{"object" => %{"inReplyTo" => _parent_id}}} = tail, %Activity{data: %{"object" => %{"inReplyTo" => parent_id}}} = tail,
user user
) do )
when is_binary(parent_id) do
parent = Activity.get_in_reply_to_activity(tail) parent = Activity.get_in_reply_to_activity(tail)
visible_for_user?(tail, user) && entire_thread_visible_for_user?(parent, user) visible_for_user?(tail, user) && entire_thread_visible_for_user?(parent, user)
end end