Don't crash ingesting >1M bytes payloads (#4872)

* Don't crash ingesting >1M bytes payloads

* Format

lib/plausible/ingestion/request.ex

@@ -112,10 +112,12 @@ defmodule Plausible.Ingestion.Request do
   defp parse_body(conn) do
     case conn.body_params do
       %Plug.Conn.Unfetched{} ->
-        {:ok, body, _conn} = Plug.Conn.read_body(conn)
+        with max_length <- conn.assigns[:read_body_limit] || 1_000_000,
-
+             {:ok, body, _conn} <-
-        case Jason.decode(body) do
+               Plug.Conn.read_body(conn, length: max_length, read_length: max_length),
-          {:ok, params} when is_map(params) -> {:ok, params}
+             {:ok, params} when is_map(params) <- Jason.decode(body) do
+          {:ok, params}
+        else
           _ -> {:error, :invalid_json}
         end
 

test/plausible/ingestion/request_test.exs

@@ -429,6 +429,31 @@ defmodule Plausible.Ingestion.RequestTest do
     assert changeset.errors[:request]
   end
 
+  test "long body length" do
+    payload = """
+    {
+      "name": "pageview", 
+      "domain": "dummy.site",
+      "url": "#{:binary.copy("a", 1_000)}"
+    }
+    """
+
+    within_read_limit =
+      :post
+      |> build_conn("/api/events", payload)
+      |> Request.build()
+
+    assert {:ok, _} = within_read_limit
+
+    exceeding_read_limit =
+      :post
+      |> build_conn("/api/events", payload)
+      |> Plug.Conn.assign(:read_body_limit, 800)
+      |> Request.build()
+
+    assert {:error, _} = exceeding_read_limit
+  end
+
   @tag :ee_only
   test "encodable" do
     params = %{