Condition de concurrence dans le script downsize_upload

concerto · Janvier 25, 2021, 7:45

Problème

Lors de l’utilisation du script downsize_upload.rb pour réduire la taille des images téléchargées, certaines images ont été modifiées, tandis que les publications qui les utilisent sont restées inchangées. Cela peut entraîner l’apparition d’images manquantes dans certaines publications.

Cause possible

Dans shrink_uploaded_image.rb, après la sélection des publications à transformer (mise à jour de l’URL de l’image), il peut arriver qu’une nouvelle publication utilise l’image téléchargée. Cependant, cette nouvelle publication n’a pas été transformée et fait toujours référence à l’ancienne version de l’image, comme illustré dans l’image ci-dessus.

Les publications nécessitant une transformation sont d’abord sélectionnées ici :

github.com/discourse/discourse

lib/shrink_uploaded_image.rb

d0ef952af


      
            end
          
            upload.url = url
          end
          
          log "base62: #{original_upload.base62_sha1} -> #{Upload.base62_sha1(sha1)}"
          log "sha: #{original_upload.sha1} -> #{sha1}"
          log "(an exisiting upload)" if existing_upload
          
          success = true
          posts = Post.unscoped.joins(:post_uploads).where(post_uploads: { upload_id: original_upload.id }).uniq.sort_by(&:created_at)
          
          posts.each do |post|
            transform_post(post, original_upload, upload)
          
            if post.custom_fields[Post::DOWNLOADED_IMAGES].present?
              downloaded_images = JSON.parse(post.custom_fields[Post::DOWNLOADED_IMAGES])
            end
          
            if post.raw_changed?
              log "Updating post"

Après ce point, toute nouvelle publication utilisant l’image en cours de transformation ne verra pas son lien mis à jour, ce qui entraînera une image brisée.

Sujet		Réponses	Vues
Downsize and Convert Image Uploads Support	1	568	Mars 17, 2022
Something I can't explain results in duplicate copies of images Support	2	518	Mars 18, 2022
Changing max_image_size_kb Support uploads , s3	5	466	Janvier 4, 2023
How can I migrate files from the old upload scheme (default/XXX) to the new one (default/original/{1,2}X)? Support	19	3237	Septembre 29, 2022
How to find any missing images? Support	62	7154	Juillet 31, 2019

Condition de concurrence dans le script downsize_upload

Problème

Cause possible

Sujets connexes