MushroomObserver / mushroom-observer / 29570361238

99%

main: 99%

Pull #4824

github

Fix mirror not busting caches: force updated_at when files rewritten

Image::Processor#update_image_record_width_height_and_transferred
relied on width/height/transferred going dirty to produce an UPDATE.
A mirror leaves width/height unchanged, and transferred can already
be false (any environment with no writable image servers, or a
not-yet-transferred image), so Rails skipped the UPDATE entirely --
updated_at never moved, the #4808 URL token never changed, and
browsers kept the pre-mirror bytes. Rotations dodged this only
because width/height swap. Force updated_at explicitly so any
reprocessing that rewrites the files always flips the token.

Found by Nathan testing #4824: mirrored dev images had fresh file
mtimes but day-old updated_at; rotated ones bumped both.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

6 of 6 new or added lines in 3 files covered. (100.0%)

52701 of 53166 relevant lines covered (99.13%)

767.16 hits per line