Skip to content

Serve cached images without regenerating them - #60

Merged
jmsche merged 1 commit into
6.xfrom
cache-hit-short-circuit
Nov 6, 2026
Merged

jmsche merged 1 commit into
6.xfrom
cache-hit-short-circuit

Conversation

@jmsche

@jmsche jmsche commented Nov 6, 2026

Copy link
Copy Markdown
Member
  • Skip ImageMagick processing & remote download when the cached file already exists
  • Use BinaryFileResponse with ETag, Last-Modified & long-lived Cache-Control headers (304 support)
  • Return a 404 for paths or formats containing ".." segments
  • Remove invalid or partially downloaded remote images from the cache
  • Add Manager::getCacheFilePath()

- Skip ImageMagick processing & remote download when the cached file already exists
- Use BinaryFileResponse with ETag, Last-Modified & long-lived Cache-Control headers (304 support)
- Return a 404 for paths or formats containing ".." segments
- Remove invalid or partially downloaded remote images from the cache
- Add Manager::getCacheFilePath()
@jmsche
jmsche merged commit de52c11 into 6.x Nov 6, 2026
10 checks passed
@jmsche
jmsche deleted the cache-hit-short-circuit branch November 6, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant