Race condition im downsize_upload Skript

concerto · 25. Januar 2021 um 07:45

Problem

Be der Verwendung des Skripts downsize_upload.rb zum Verkleinern hochgeladener Bilder wurden einige Bilder geändert, während die Beiträge, die diese Bilder verwenden, unverändert blieben. Dies kann dazu führen, dass Bilder in einigen Beiträgen so aussehen, als wären sie verschwunden.

Mögliche Ursache

In shrink_uploaded_image.rb kann es nach der Auswahl der zu transformierenden Beiträge (Aktualisierung der Bild-URL) vorkommen, dass ein neuer Beitrag das hochgeladene Bild verwendet. Dieser neue Beitrag wurde jedoch nicht transformiert und verweist weiterhin auf eine alte Referenz des Bildes, wie im Bild oben gezeigt.

Die zu transformierenden Beiträge werden hier zunächst ausgewählt:

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"

Nach diesem Punkt werden Links in neuen Beiträgen, die das transformierte Bild verwenden, nicht aktualisiert, was zu einem defekten Bild führt.

Thema		Antworten	Aufrufe
Downsize and Convert Image Uploads Support	1	568	17. März 2022
Something I can't explain results in duplicate copies of images Support	2	518	18. März 2022
Changing max_image_size_kb Support uploads , s3	5	466	4. Januar 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	29. September 2022
How to find any missing images? Support	62	7154	31. Juli 2019

Race condition im downsize_upload Skript

Problem

Mögliche Ursache

Verwandte Themen