tommy/mobilizon
1
0
Fork 0
forked from potsda.mn/mobilizon
Code Pull requests Activity

Add details of object in exception when AP object as no

Browse source 
actor/attributedTo

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2022-03-29 10:39:32 +02:00
parent bd64e19af4
commit 3c8a2a1313
No known key found for this signature in database
GPG key ID: A061B9DDE0CA0773
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/federation/activity_pub/utils.ex
View file

@ -337,8 +337,9 @@ defmodule Mobilizon.Federation.ActivityPub.Utils do
message: "Object contains an actor object with invalid type: #{inspect(type)}"
end
def get_actor(%{"actor" => nil, "attributedTo" => nil}) do
raise ArgumentError, message: "Object contains both actor and attributedTo fields being null"
def get_actor(%{"actor" => nil, "attributedTo" => nil} = object) do
raise ArgumentError,
message: "Object contains both actor and attributedTo fields being null: #{inspect(object)}"
end
def get_actor(%{"actor" => _}) do

2
test/federation/activity_pub/utils_test.exs
View file

@ -129,7 +129,7 @@ defmodule Mobilizon.Federation.ActivityPub.UtilsTest do
test "with no actor information" do
assert_raise ArgumentError,
"Object contains both actor and attributedTo fields being null",
"Object contains both actor and attributedTo fields being null: %{\"actor\" => nil, \"attributedTo\" => nil}",
fn ->
Utils.get_actor(%{
"actor" => nil,