From 3428ad4537606cf3803d0695690d1d826668dca4 Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Thu, 18 Sep 2025 20:06:34 +0300
Subject: [PATCH] Serve artist images from assets

---
 .../controllers/artist_controller.ex          | 30 +++++--------------
 1 file changed, 8 insertions(+), 22 deletions(-)

diff --git a/lib/music_library_web/controllers/artist_controller.ex b/lib/music_library_web/controllers/artist_controller.ex
index 0f63b680..662cddd6 100644
--- a/lib/music_library_web/controllers/artist_controller.ex
+++ b/lib/music_library_web/controllers/artist_controller.ex
@@ -2,20 +2,20 @@ defmodule MusicLibraryWeb.ArtistController do
   use MusicLibraryWeb, :controller
 
   alias MusicLibrary.Artists
-
-  # 1 year in seconds
-  @cache_duration 60 * 60 * 24 * 365
+  alias MusicLibrary.Assets.Transform
+  alias MusicLibraryWeb.CoverController
 
   def image(conn, %{"musicbrainz_id" => artist_id}) do
     case Artists.get_image(artist_id) do
       nil ->
         not_found(conn)
 
-      %{image_data: image_data, image_data_hash: etag} ->
-        case get_req_header(conn, "if-none-match") do
-          [^etag] -> extend_cache(conn)
-          _ -> respond_with_cache(conn, image_data, etag)
-        end
+      %{image_data_hash: hash} ->
+        payload =
+          %Transform{hash: hash}
+          |> Transform.encode!()
+
+        CoverController.show(conn, %{"transform_payload" => payload})
     end
   end
 
@@ -24,18 +24,4 @@ defmodule MusicLibraryWeb.ArtistController do
     |> put_status(:not_found)
     |> text("Not found")
   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