[PR #3584] [MERGED] Improve author image performance #4024

Closed
opened 2026-04-25 00:18:01 +02:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/advplyr/audiobookshelf/pull/3584
Author: @mikiher
Created: 11/3/2024
Status: Merged
Merged: 11/3/2024
Merged by: @advplyr

Base: masterHead: author-image-performance


📝 Commits (3)

  • 3bc2941 No db access for author image if in disk cache
  • bf84072 No auth for author images
  • 68fd1d6 Remove token from author image URLs

📊 Changes

5 files changed (+29 additions, -17 deletions)

View changed files

📝 client/components/covers/AuthorImage.vue (+1 -1)
📝 server/Auth.js (+2 -2)
📝 server/controllers/AuthorController.js (+14 -7)
📝 server/managers/CacheManager.js (+11 -6)
📝 server/routers/ApiRouter.js (+1 -1)

📄 Description

This is the same as PR #3580, for author images.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/advplyr/audiobookshelf/pull/3584 **Author:** [@mikiher](https://github.com/mikiher) **Created:** 11/3/2024 **Status:** ✅ Merged **Merged:** 11/3/2024 **Merged by:** [@advplyr](https://github.com/advplyr) **Base:** `master` ← **Head:** `author-image-performance` --- ### 📝 Commits (3) - [`3bc2941`](https://github.com/advplyr/audiobookshelf/commit/3bc29414456b60d0653d32b9a3fc72c88c3b2400) No db access for author image if in disk cache - [`bf84072`](https://github.com/advplyr/audiobookshelf/commit/bf8407274e3ee300af1927ee660d078a7a801e1c) No auth for author images - [`68fd1d6`](https://github.com/advplyr/audiobookshelf/commit/68fd1d67cb867c1deeb0cac772dede2935a63527) Remove token from author image URLs ### 📊 Changes **5 files changed** (+29 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `client/components/covers/AuthorImage.vue` (+1 -1) 📝 `server/Auth.js` (+2 -2) 📝 `server/controllers/AuthorController.js` (+14 -7) 📝 `server/managers/CacheManager.js` (+11 -6) 📝 `server/routers/ApiRouter.js` (+1 -1) </details> ### 📄 Description This is the same as PR #3580, for author images. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2026-04-25 00:18:01 +02:00
adam closed this issue 2026-04-25 00:18:01 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/audiobookshelf#4024