From 73a943bc632e383138b4297dea74f80e820f780e Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Thu, 5 Mar 2026 10:50:35 +0000
Subject: [PATCH] Server records on this day email assets from a public
 endpoint

---
 docs/architecture.md                           |  2 +-
 lib/music_library/records_on_this_day_email.ex | 13 ++++++-------
 lib/music_library_web/router.ex                |  2 ++
 3 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/docs/architecture.md b/docs/architecture.md
index 215119a6..d31e8c1c 100644
--- a/docs/architecture.md
+++ b/docs/architecture.md
@@ -288,7 +288,7 @@ All authenticated routes live inside a single `live_session` with three `on_moun
 | `HealthController` | `/health` | Health check |
 | `LastFmController` | `/auth/last_fm/callback` | Last.fm OAuth |
 | `ArchiveController` | `/backup`, `/api/backup` | Database backup download (API route requires token) |
-| `AssetController` | `/assets/:transform_payload`, `/api/assets/:transform_payload` | Serve images with transforms (API route requires token) |
+| `AssetController` | `/assets/:transform_payload`, `/public/assets/:transform_payload`, `/api/assets/:transform_payload` | Serve images with transforms (public route for emails, API route requires token) |
 | `CollectionController` | `/api/collection/*` | JSON API for collection queries |
 
 ---
diff --git a/lib/music_library/records_on_this_day_email.ex b/lib/music_library/records_on_this_day_email.ex
index 107623c4..3eb23b42 100644
--- a/lib/music_library/records_on_this_day_email.ex
+++ b/lib/music_library/records_on_this_day_email.ex
@@ -44,11 +44,10 @@ defmodule MusicLibrary.RecordsOnThisDayEmail do
 
   defp build_html(records, date, conf) do
     base_url = Keyword.fetch!(conf, :base_url) |> String.trim_trailing("/")
-    api_token = Application.get_env(:music_library, MusicLibraryWeb)[:api_token]
     heading = date |> Calendar.strftime("Records on %-d %B") |> html_escape()
 
     records_html =
-      Enum.map_join(records, "\n", fn record -> record_html(record, date, base_url, api_token) end)
+      Enum.map_join(records, "\n", fn record -> record_html(record, date, base_url) end)
 
     """
     <div style="max-width: 600px; margin: 0 auto; padding: 20px; font-family: system-ui, -apple-system, sans-serif; background-color: #f4f4f5;">
@@ -62,9 +61,9 @@ defmodule MusicLibrary.RecordsOnThisDayEmail do
     """
   end
 
-  defp record_html(record, date, base_url, api_token) do
+  defp record_html(record, date, base_url) do
     years = Record.released_how_long_ago?(record, date)
-    cover_url = cover_image_url(record, base_url, api_token)
+    cover_url = cover_image_url(record, base_url)
     record_url = record_detail_url(record, base_url)
     artist_names = record |> Record.artist_names() |> html_escape()
     title = record.title |> html_escape()
@@ -106,11 +105,11 @@ defmodule MusicLibrary.RecordsOnThisDayEmail do
     "#{base_url}/collection/#{record.id}"
   end
 
-  defp cover_image_url(%{cover_hash: nil}, _base_url, _api_token), do: nil
+  defp cover_image_url(%{cover_hash: nil}, _base_url), do: nil
 
-  defp cover_image_url(record, base_url, api_token) do
+  defp cover_image_url(record, base_url) do
     payload = Transform.new(hash: record.cover_hash, width: 96) |> Transform.encode!()
-    "#{base_url}/api/assets/#{payload}?token=#{api_token}"
+    "#{base_url}/public/assets/#{payload}"
   end
 
   defp years_ago_label(nil), do: ""
diff --git a/lib/music_library_web/router.ex b/lib/music_library_web/router.ex
index 2f57b2a8..fab1e260 100644
--- a/lib/music_library_web/router.ex
+++ b/lib/music_library_web/router.ex
@@ -32,6 +32,8 @@ defmodule MusicLibraryWeb.Router do
 
     get "/auth/last_fm/callback", LastFmController, :callback
 
+    get "/public/assets/:transform_payload", AssetController, :show
+
     scope "/" do
       pipe_through :logged_in