Don't expire idempotency cache for five minutes.

This commit is contained in:
lain 2018-05-04 21:15:39 +02:00
parent fcd3eca167
commit 90c4bed0af
2 changed files with 6 additions and 0 deletions

View File

@ -288,6 +288,8 @@ def post_status(%{assigns: %{user: user}} = conn, %{"status" => _} = params) do
fallback: fn _ -> CommonAPI.post(user, params) end fallback: fn _ -> CommonAPI.post(user, params) end
) )
Cachex.expire(:user_cache, "idem:#{idempotency_key}", :timer.seconds(5 * 60))
render(conn, StatusView, "status.json", %{activity: activity, for: user, as: :activity}) render(conn, StatusView, "status.json", %{activity: activity, for: user, as: :activity})
end end

View File

@ -75,6 +75,10 @@ test "posting a status", %{conn: conn} do
"sensitive" => "false" "sensitive" => "false"
}) })
{:ok, ttl} = Cachex.ttl(:user_cache, "idem:#{idempotency_key}")
# 5 Minutes
assert ttl > :timer.seconds(5 * 60 - 1)
assert %{"content" => "cofe", "id" => id, "spoiler_text" => "2hu", "sensitive" => false} = assert %{"content" => "cofe", "id" => id, "spoiler_text" => "2hu", "sensitive" => false} =
json_response(conn_one, 200) json_response(conn_one, 200)