Serve artist images from assets
This commit is contained in:
@@ -2,20 +2,20 @@ defmodule MusicLibraryWeb.ArtistController do
|
|||||||
use MusicLibraryWeb, :controller
|
use MusicLibraryWeb, :controller
|
||||||
|
|
||||||
alias MusicLibrary.Artists
|
alias MusicLibrary.Artists
|
||||||
|
alias MusicLibrary.Assets.Transform
|
||||||
# 1 year in seconds
|
alias MusicLibraryWeb.CoverController
|
||||||
@cache_duration 60 * 60 * 24 * 365
|
|
||||||
|
|
||||||
def image(conn, %{"musicbrainz_id" => artist_id}) do
|
def image(conn, %{"musicbrainz_id" => artist_id}) do
|
||||||
case Artists.get_image(artist_id) do
|
case Artists.get_image(artist_id) do
|
||||||
nil ->
|
nil ->
|
||||||
not_found(conn)
|
not_found(conn)
|
||||||
|
|
||||||
%{image_data: image_data, image_data_hash: etag} ->
|
%{image_data_hash: hash} ->
|
||||||
case get_req_header(conn, "if-none-match") do
|
payload =
|
||||||
[^etag] -> extend_cache(conn)
|
%Transform{hash: hash}
|
||||||
_ -> respond_with_cache(conn, image_data, etag)
|
|> Transform.encode!()
|
||||||
end
|
|
||||||
|
CoverController.show(conn, %{"transform_payload" => payload})
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@@ -24,18 +24,4 @@ defmodule MusicLibraryWeb.ArtistController do
|
|||||||
|> put_status(:not_found)
|
|> put_status(:not_found)
|
||||||
|> text("Not found")
|
|> text("Not found")
|
||||||
end
|
end
|
||||||
|
|
||||||
defp extend_cache(conn) do
|
|
||||||
conn
|
|
||||||
|> put_resp_header("cache-control", "public, max-age=#{@cache_duration}")
|
|
||||||
|> send_resp(304, "")
|
|
||||||
end
|
|
||||||
|
|
||||||
defp respond_with_cache(conn, image_data, etag) do
|
|
||||||
conn
|
|
||||||
|> put_resp_content_type("image/jpeg", "utf-8")
|
|
||||||
|> put_resp_header("cache-control", "public, max-age=#{@cache_duration}")
|
|
||||||
|> put_resp_header("etag", etag)
|
|
||||||
|> send_resp(200, image_data)
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user