Fix error when trying to delete already-deleted file with OpenStack S…

…wift (#27569)
mastodon · Nov 27, 2023 · d01d51d · d01d51d
1 parent 3c33d79
commit d01d51d
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/app/lib/attachment_batch.rb b/app/lib/attachment_batch.rb
@@ -75,7 +75,12 @@ def remove_files
             end
           when :fog
             logger.debug { "Deleting #{attachment.path(style)}" }
-            attachment.send(:directory).files.new(key: attachment.path(style)).destroy
+
+            begin
+              attachment.send(:directory).files.new(key: attachment.path(style)).destroy
+            rescue Fog::Storage::OpenStack::NotFound
+              # Ignore failure to delete a file that has already been deleted
+            end
           when :azure
             logger.debug { "Deleting #{attachment.path(style)}" }
             attachment.destroy