Merge branch 'fix/apc2s-limits' into 'develop'

AP C2S: Restrict character limit on Note

Closes #2

See merge request pleroma/secteam/pleroma!9

lib/pleroma/web/activity_pub/activity_pub_controller.ex

@@ -399,21 +399,30 @@ def read_inbox(%{assigns: %{user: %User{nickname: as_nickname}}} = conn, %{
 
   defp handle_user_activity(
          %User{} = user,
-         %{"type" => "Create", "object" => %{"type" => "Note"}} = params
+         %{"type" => "Create", "object" => %{"type" => "Note"} = object} = params
        ) do
-    object =
+    content = if is_binary(object["content"]), do: object["content"], else: ""
-      params["object"]
+    name = if is_binary(object["name"]), do: object["name"], else: ""
-      |> Map.merge(Map.take(params, ["to", "cc"]))
+    summary = if is_binary(object["summary"]), do: object["summary"], else: ""
-      |> Map.put("attributedTo", user.ap_id())
+    length = String.length(content <> name <> summary)
-      |> Transmogrifier.fix_object()
 
-    ActivityPub.create(%{
+    if length > Pleroma.Config.get([:instance, :limit]) do
-      to: params["to"],
+      {:error, dgettext("errors", "Note is over the character limit")}
-      actor: user,
+    else
-      context: object["context"],
+      object =
-      object: object,
+        object
-      additional: Map.take(params, ["cc"])
+        |> Map.merge(Map.take(params, ["to", "cc"]))
-    })
+        |> Map.put("attributedTo", user.ap_id())
+        |> Transmogrifier.fix_object()
+
+      ActivityPub.create(%{
+        to: params["to"],
+        actor: user,
+        context: object["context"],
+        object: object,
+        additional: Map.take(params, ["cc"])
+      })
+    end
   end
 
   defp handle_user_activity(%User{} = user, %{"type" => "Delete"} = params) do

test/web/activity_pub/activity_pub_controller_test.exs

@@ -905,6 +905,8 @@ test "it requires authentication if instance is NOT federating", %{
   end
 
   describe "POST /users/:nickname/outbox (C2S)" do
+    setup do: clear_config([:instance, :limit])
+
     setup do
       [
         activity: %{
@@ -1121,6 +1123,20 @@ test "it doesn't spreads faulty attributedTo or actor fields", %{
       assert cirno_object.data["actor"] == cirno.ap_id
       assert cirno_object.data["attributedTo"] == cirno.ap_id
     end
+
+    test "Character limitation", %{conn: conn, activity: activity} do
+      Pleroma.Config.put([:instance, :limit], 5)
+      user = insert(:user)
+
+      result =
+        conn
+        |> assign(:user, user)
+        |> put_req_header("content-type", "application/activity+json")
+        |> post("/users/#{user.nickname}/outbox", activity)
+        |> json_response(400)
+
+      assert result == "Note is over the character limit"
+    end
   end
 
   describe "/relay/followers" do