In Ruby on Rails, After send_file method delete the file from server
Solution 1
Because you're using send_file
, Rails will pass the request along to your HTTP server (nginx, apache, etc. - See the Rails documentation on send_file regarding X-Sendfile headers). Because of this, when you try to delete the file, Rails doesn't know that it's still being used.
You can try using send_data
instead, which will block until the data is sent, allowing your File.delete
request to succeed. Keep in mind that send_data
requires a data stream as its argument though, not a path, so you need to open the file first:
File.open(file_path, 'r') do |f|
send_data f.read, type: "text/excel"
end
File.delete(file_path)
The other option would be a background job that periodically checks for temp files to delete.
Solution 2
If you are generating on the fly the file you are trying to send, a solution is to use the Tempfile
class:
begin
# The second argument is optional:
temp_file = Tempfile.new(filename, temp_directory)
# ... edit temp_file as needed.
# By default, temp files are 0600,
# so this line might be needed depending on your configuration:
temp_file.chmod(0644)
send_file temp_file
ensure
temp_file.close
end
Contrary to what is indicated in this question, this works as expected (the file stays on the server long enough to be served, but ultimately gets deleted); this post seems to indicate this is due to updates in Rails 3.2.11, something I couldn’t verify.
Solution 3
This works for me!
With send_data
you can delete file before send.
file = File.open(Rails.root.join('public', 'uploads', filename), "rb")
contents = file.read
file.close
File.delete(filepath) if File.exist?(filepath)
send_data(contents, :filename => filename)
Chetan Kalore
Updated on June 17, 2022Comments
-
Chetan Kalore almost 2 years
I am using following code for sending the file in Rails.
if File.exist?(file_path) send_file(file_path, type: 'text/excel') File.delete(file_path) end
In this I am trying to send the file and delete the file from server once it is been send successfully. But I am facing issue is, the delete operation is getting executed while sending is performing and due to that I don't see anything in browser.
So is there any way in Rails, once the send_file operation is completed delete the file from server.
Any help on this would be highly appreciated.
Thanks,
Chetan -
Neil Slater almost 11 yearsAlso keep in mind that send_data will likely use more resources to send same content as
send_file
, but no need to obsess about the difference unless this will be a heavily-used feature of the site. -
Dylan Markow almost 11 years@NeilSlater Yes, for a basic text file that generates quickly it shouldn't be a big deal. But for generating large/slow files, a send_file + background job solution would probably be better.
-
Chetan Kalore almost 11 yearsThanks Dylan for responding on this. Following worked for me: if File.exist?(file_path) File.open(file_path, 'rb') do |f| send_data f.read, type: "application/excel", filename: "file_name.xls" end end
-
Michael Wu almost 10 yearsI'm pretty sure if you want to actually delete the tempfile, you need to do something like temp_file.unlink
-
louije almost 10 years@MichaelWu According to the documentation, TempFile#close with no parameter ends up calling #unlink automatically, "when the object is finalized". It worked as expected for me.
-
mrbrdo about 9 yearsI wouldn't count on this working 100% of the time. It will delete the file when the Tempfile is garbage collected, and there are no guarantees that this will happen before nginx/apache has opened the file. It depends on the internal implementation of send_file in Rails. send_data is a better alternative.
-
Taimoor Changaiz over 8 yearsThanks Dylan for simple solution (y)