Use a temporary folder

madeindjs · Dec 3, 2018 · 9a2ad36 · 9a2ad36
1 parent 59bbef6
commit 9a2ad36
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/active_storage/send_zip.rb b/lib/active_storage/send_zip.rb
@@ -1,6 +1,7 @@
 require 'active_storage/send_zip/version'
 require 'rails'
 require 'zip'
+require 'tempfile'
 
 module ActiveStorage
   module SendZip
@@ -29,8 +30,8 @@ def send_zip(active_storages, filename: 'my.zip')
     def save_files_on_server(files)
       require 'zip'
       # get a temporary folder and create it
-      temp_folder = File.join(Dir.tmpdir, 'user')
-      FileUtils.mkdir_p(temp_folder) unless Dir.exist?(temp_folder)
+      temp_folder = Dir.mktmpdir 'active_storage-send_zip'
+      # FileUtils.mkdir_p(temp_folder) unless Dir.exist?(temp_folder)
 
       # download all ActiveStorage into
       files.map do |picture|