Inbox: Don't add the same thing twice.
authorlain <lain@soykaf.club>
Sun, 18 Feb 2018 21:37:44 +0000 (22:37 +0100)
committerlain <lain@soykaf.club>
Sun, 18 Feb 2018 21:37:44 +0000 (22:37 +0100)
lib/pleroma/web/activity_pub/activity_pub_controller.ex

index f4e5ae9eb32699d0312ef01bdc3fbbc7e8f82cfe..da4973fe571e7e4db431c4844b9ed615658b50f2 100644 (file)
@@ -1,6 +1,6 @@
 defmodule Pleroma.Web.ActivityPub.ActivityPubController do
   use Pleroma.Web, :controller
-  alias Pleroma.{User, Repo, Object}
+  alias Pleroma.{User, Repo, Object, Activity}
   alias Pleroma.Web.ActivityPub.{ObjectView, UserView, Transmogrifier}
   alias Pleroma.Web.ActivityPub.ActivityPub
 
@@ -26,9 +26,13 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubController do
   def inbox(%{assigns: %{valid_signature: true}} = conn, params) do
     # File.write("/tmp/incoming.json", Poison.encode!(params))
     with {:ok, _user} <- ap_enabled_actor(params["actor"]),
+         nil <- Activity.get_by_ap_id(params["id"]),
          {:ok, activity} <- Transmogrifier.handle_incoming(params) do
       json(conn, "ok")
     else
+      %Activity{} ->
+        Logger.info("Already had #{params["id"]}")
+        json(conn, "ok")
       e ->
         # Just drop those for now
         Logger.info("Unhandled activity")