Merge branch 'feature/federation-job-insert-connection-checkout' into 'develop'

Publisher: check out a connection for inserting publish_one jobs

See merge request pleroma/pleroma!2063

lib/pleroma/web/activity_pub/publisher.ex

@@ -9,6 +9,7 @@ defmodule Pleroma.Web.ActivityPub.Publisher do
   alias Pleroma.HTTP
   alias Pleroma.Instances
   alias Pleroma.Object
+  alias Pleroma.Repo
   alias Pleroma.User
   alias Pleroma.Web.ActivityPub.Relay
   alias Pleroma.Web.ActivityPub.Transmogrifier
@@ -188,31 +189,35 @@ def publish(%User{} = actor, %{data: %{"bcc" => bcc}} = activity)
 
     recipients = recipients(actor, activity)
 
-    recipients
+    inboxes =
-    |> Enum.filter(&User.ap_enabled?/1)
+      recipients
-    |> Enum.map(fn %{source_data: data} -> data["inbox"] end)
+      |> Enum.filter(&User.ap_enabled?/1)
-    |> Enum.filter(fn inbox -> should_federate?(inbox, public) end)
+      |> Enum.map(fn %{source_data: data} -> data["inbox"] end)
-    |> Instances.filter_reachable()
+      |> Enum.filter(fn inbox -> should_federate?(inbox, public) end)
-    |> Enum.each(fn {inbox, unreachable_since} ->
+      |> Instances.filter_reachable()
-      %User{ap_id: ap_id} =
-        Enum.find(recipients, fn %{source_data: data} -> data["inbox"] == inbox end)
 
-      # Get all the recipients on the same host and add them to cc. Otherwise, a remote
+    Repo.checkout(fn ->
-      # instance would only accept a first message for the first recipient and ignore the rest.
+      Enum.each(inboxes, fn {inbox, unreachable_since} ->
-      cc = get_cc_ap_ids(ap_id, recipients)
+        %User{ap_id: ap_id} =
+          Enum.find(recipients, fn %{source_data: data} -> data["inbox"] == inbox end)
 
-      json =
+        # Get all the recipients on the same host and add them to cc. Otherwise, a remote
-        data
+        # instance would only accept a first message for the first recipient and ignore the rest.
-        |> Map.put("cc", cc)
+        cc = get_cc_ap_ids(ap_id, recipients)
-        |> Jason.encode!()
 
-      Pleroma.Web.Federator.Publisher.enqueue_one(__MODULE__, %{
+        json =
-        inbox: inbox,
+          data
-        json: json,
+          |> Map.put("cc", cc)
-        actor_id: actor.id,
+          |> Jason.encode!()
-        id: activity.data["id"],
+
-        unreachable_since: unreachable_since
+        Pleroma.Web.Federator.Publisher.enqueue_one(__MODULE__, %{
-      })
+          inbox: inbox,
+          json: json,
+          actor_id: actor.id,
+          id: activity.data["id"],
+          unreachable_since: unreachable_since
+        })
+      end)
     end)
   end