Allow to skip cache in Cache plug
Ref: fix-local-public
This commit is contained in:
parent
0fe674d898
commit
370a241dbc
2 changed files with 30 additions and 7 deletions
|
@ -98,14 +98,19 @@ defp cache_resp(conn, opts) do
|
|||
content_type = content_type(conn)
|
||||
|
||||
conn =
|
||||
unless opts[:tracking_fun] do
|
||||
@cachex.put(:web_resp_cache, key, {content_type, body}, ttl: ttl)
|
||||
conn
|
||||
else
|
||||
tracking_fun_data = Map.get(conn.assigns, :tracking_fun_data, nil)
|
||||
@cachex.put(:web_resp_cache, key, {content_type, body, tracking_fun_data}, ttl: ttl)
|
||||
cond do
|
||||
Map.get(conn.assigns, :skip_cache, false) ->
|
||||
conn
|
||||
|
||||
opts.tracking_fun.(conn, tracking_fun_data)
|
||||
!opts[:tracking_fun] ->
|
||||
@cachex.put(:web_resp_cache, key, {content_type, body}, ttl: ttl)
|
||||
conn
|
||||
|
||||
true ->
|
||||
tracking_fun_data = Map.get(conn.assigns, :tracking_fun_data, nil)
|
||||
@cachex.put(:web_resp_cache, key, {content_type, body, tracking_fun_data}, ttl: ttl)
|
||||
|
||||
opts.tracking_fun.(conn, tracking_fun_data)
|
||||
end
|
||||
|
||||
put_resp_header(conn, "x-cache", "MISS from Pleroma")
|
||||
|
|
|
@ -179,4 +179,22 @@ test "ignore non-successful responses" do
|
|||
|> send_resp(:im_a_teapot, "🥤")
|
||||
|> sent_resp()
|
||||
end
|
||||
|
||||
test "ignores if skip_cache is assigned" do
|
||||
assert @miss_resp ==
|
||||
conn(:get, "/")
|
||||
|> assign(:skip_cache, true)
|
||||
|> Cache.call(%{query_params: false, ttl: nil})
|
||||
|> put_resp_content_type("cofe/hot")
|
||||
|> send_resp(:ok, "cofe")
|
||||
|> sent_resp()
|
||||
|
||||
assert @miss_resp ==
|
||||
conn(:get, "/")
|
||||
|> assign(:skip_cache, true)
|
||||
|> Cache.call(%{query_params: false, ttl: nil})
|
||||
|> put_resp_content_type("cofe/hot")
|
||||
|> send_resp(:ok, "cofe")
|
||||
|> sent_resp()
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue