Cache covers for 1 year
This commit is contained in:
@@ -4,8 +4,8 @@ defmodule MusicLibraryWeb.CoverController do
|
|||||||
alias MusicLibrary.Records
|
alias MusicLibrary.Records
|
||||||
alias MusicLibrary.Records.Cover
|
alias MusicLibrary.Records.Cover
|
||||||
|
|
||||||
# 24 hours in seconds
|
# 1 year in seconds
|
||||||
@cache_duration 60 * 60 * 24
|
@cache_duration 60 * 60 * 24 * 365
|
||||||
|
|
||||||
def show(conn, %{"record_id" => record_id, "size" => size}) do
|
def show(conn, %{"record_id" => record_id, "size" => size}) do
|
||||||
case Records.get_cover(record_id) do
|
case Records.get_cover(record_id) do
|
||||||
|
|||||||
@@ -23,7 +23,7 @@ defmodule MusicLibraryWeb.CoverControllerTest do
|
|||||||
|
|
||||||
assert conn.status == 200
|
assert conn.status == 200
|
||||||
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
||||||
assert get_resp_header(conn, "cache-control") == ["public, max-age=86400"]
|
assert get_resp_header(conn, "cache-control") == ["public, max-age=31536000"]
|
||||||
assert get_resp_header(conn, "etag") == [record.cover_hash]
|
assert get_resp_header(conn, "etag") == [record.cover_hash]
|
||||||
|
|
||||||
assert conn.resp_body == record.cover_data
|
assert conn.resp_body == record.cover_data
|
||||||
@@ -37,7 +37,7 @@ defmodule MusicLibraryWeb.CoverControllerTest do
|
|||||||
|
|
||||||
assert conn.status == 200
|
assert conn.status == 200
|
||||||
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
||||||
assert get_resp_header(conn, "cache-control") == ["public, max-age=86400"]
|
assert get_resp_header(conn, "cache-control") == ["public, max-age=31536000"]
|
||||||
assert get_resp_header(conn, "etag") == [record.cover_hash]
|
assert get_resp_header(conn, "etag") == [record.cover_hash]
|
||||||
|
|
||||||
assert conn.resp_body == record.cover_data
|
assert conn.resp_body == record.cover_data
|
||||||
@@ -51,7 +51,7 @@ defmodule MusicLibraryWeb.CoverControllerTest do
|
|||||||
|
|
||||||
assert conn.status == 304
|
assert conn.status == 304
|
||||||
assert get_resp_header(conn, "content-type") == []
|
assert get_resp_header(conn, "content-type") == []
|
||||||
assert get_resp_header(conn, "cache-control") == ["public, max-age=86400"]
|
assert get_resp_header(conn, "cache-control") == ["public, max-age=31536000"]
|
||||||
assert get_resp_header(conn, "etag") == []
|
assert get_resp_header(conn, "etag") == []
|
||||||
|
|
||||||
assert conn.resp_body == <<>>
|
assert conn.resp_body == <<>>
|
||||||
@@ -65,7 +65,7 @@ defmodule MusicLibraryWeb.CoverControllerTest do
|
|||||||
|
|
||||||
assert conn.status == 200
|
assert conn.status == 200
|
||||||
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
assert get_resp_header(conn, "content-type") == ["image/jpeg; charset=utf-8"]
|
||||||
assert get_resp_header(conn, "cache-control") == ["public, max-age=86400"]
|
assert get_resp_header(conn, "cache-control") == ["public, max-age=31536000"]
|
||||||
assert get_resp_header(conn, "etag") == [hash]
|
assert get_resp_header(conn, "etag") == [hash]
|
||||||
|
|
||||||
assert conn.resp_body == thumb
|
assert conn.resp_body == thumb
|
||||||
|
|||||||
Reference in New Issue
Block a user