Generates contexts and ids on insertion time.
This commit is contained in:
parent
a4db3a732f
commit
bc31bee7c4
@ -45,6 +45,18 @@ defmodule Pleroma.Web.ActivityPub.Utils do
|
|||||||
"#{Web.base_url()}/#{type}/#{UUID.generate()}"
|
"#{Web.base_url()}/#{type}/#{UUID.generate()}"
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def create_context(context) do
|
||||||
|
context = context || generate_id("contexts")
|
||||||
|
changeset = Object.context_mapping(context)
|
||||||
|
|
||||||
|
case Repo.insert(changeset) do
|
||||||
|
{:ok, object} -> object
|
||||||
|
# This should be solved by an upsert, but it seems ecto
|
||||||
|
# has problems accessing the constraint inside the jsonb.
|
||||||
|
{:error, _} -> Object.get_cached_by_ap_id(context)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
@doc """
|
@doc """
|
||||||
Enqueues an activity for federation if it's local
|
Enqueues an activity for federation if it's local
|
||||||
"""
|
"""
|
||||||
@ -67,13 +79,17 @@ defmodule Pleroma.Web.ActivityPub.Utils do
|
|||||||
also adds it to an included object
|
also adds it to an included object
|
||||||
"""
|
"""
|
||||||
def lazy_put_activity_defaults(map) do
|
def lazy_put_activity_defaults(map) do
|
||||||
|
%{data: %{"id" => context}, id: context_id} = create_context(map["context"])
|
||||||
|
|
||||||
map =
|
map =
|
||||||
map
|
map
|
||||||
|> Map.put_new_lazy("id", &generate_activity_id/0)
|
|> Map.put_new_lazy("id", &generate_activity_id/0)
|
||||||
|> Map.put_new_lazy("published", &make_date/0)
|
|> Map.put_new_lazy("published", &make_date/0)
|
||||||
|
|> Map.put_new("context", context)
|
||||||
|
|> Map.put_new("context_id", context_id)
|
||||||
|
|
||||||
if is_map(map["object"]) do
|
if is_map(map["object"]) do
|
||||||
object = lazy_put_object_defaults(map["object"])
|
object = lazy_put_object_defaults(map["object"], map)
|
||||||
%{map | "object" => object}
|
%{map | "object" => object}
|
||||||
else
|
else
|
||||||
map
|
map
|
||||||
@ -83,10 +99,12 @@ defmodule Pleroma.Web.ActivityPub.Utils do
|
|||||||
@doc """
|
@doc """
|
||||||
Adds an id and published date if they aren't there.
|
Adds an id and published date if they aren't there.
|
||||||
"""
|
"""
|
||||||
def lazy_put_object_defaults(map) do
|
def lazy_put_object_defaults(map, activity \\ %{}) do
|
||||||
map
|
map
|
||||||
|> Map.put_new_lazy("id", &generate_object_id/0)
|
|> Map.put_new_lazy("id", &generate_object_id/0)
|
||||||
|> Map.put_new_lazy("published", &make_date/0)
|
|> Map.put_new_lazy("published", &make_date/0)
|
||||||
|
|> Map.put_new("context", activity["context"])
|
||||||
|
|> Map.put_new("context_id", activity["context_id"])
|
||||||
end
|
end
|
||||||
|
|
||||||
@doc """
|
@doc """
|
||||||
|
@ -32,13 +32,14 @@ defmodule Pleroma.Web.TwitterAPI.ActivityView do
|
|||||||
end)
|
end)
|
||||||
end
|
end
|
||||||
|
|
||||||
defp get_context_id(%{data: %{"context" => nil}}), do: nil
|
defp get_context_id(%{data: %{"context" => nil}}, _), do: nil
|
||||||
defp get_context_id(%{data: %{"context" => context}}, options) do
|
defp get_context_id(%{data: %{"context" => context}}, options) do
|
||||||
cond do
|
cond do
|
||||||
id = options[:context_ids][context] -> id
|
id = options[:context_ids][context] -> id
|
||||||
true -> TwitterAPI.context_to_conversation_id(context)
|
true -> TwitterAPI.context_to_conversation_id(context)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
defp get_context_id(_, _), do: nil
|
||||||
|
|
||||||
def render("index.json", opts) do
|
def render("index.json", opts) do
|
||||||
context_ids = collect_context_ids(opts.activities)
|
context_ids = collect_context_ids(opts.activities)
|
||||||
|
@ -40,12 +40,31 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubTest do
|
|||||||
|
|
||||||
data = %{
|
data = %{
|
||||||
"ok" => true,
|
"ok" => true,
|
||||||
"id" => given_id
|
"id" => given_id,
|
||||||
|
"context" => "blabla"
|
||||||
}
|
}
|
||||||
|
|
||||||
{:ok, %Activity{} = activity} = ActivityPub.insert(data)
|
{:ok, %Activity{} = activity} = ActivityPub.insert(data)
|
||||||
assert activity.data["ok"] == data["ok"]
|
assert activity.data["ok"] == data["ok"]
|
||||||
assert activity.data["id"] == given_id
|
assert activity.data["id"] == given_id
|
||||||
|
assert activity.data["context"] == "blabla"
|
||||||
|
assert activity.data["context_id"]
|
||||||
|
end
|
||||||
|
|
||||||
|
test "adds a context when none is there" do
|
||||||
|
data = %{
|
||||||
|
"id" => "some_id",
|
||||||
|
"object" => %{
|
||||||
|
"id" => "object_id"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
{:ok, %Activity{} = activity} = ActivityPub.insert(data)
|
||||||
|
|
||||||
|
assert is_binary(activity.data["context"])
|
||||||
|
assert is_binary(activity.data["object"]["context"])
|
||||||
|
assert activity.data["context_id"]
|
||||||
|
assert activity.data["object"]["context_id"]
|
||||||
end
|
end
|
||||||
|
|
||||||
test "adds an id to a given object if it lacks one and is a note and inserts it to the object database" do
|
test "adds an id to a given object if it lacks one and is a note and inserts it to the object database" do
|
||||||
|
Loading…
Reference in New Issue
Block a user