4

我正在尝试使用 OpenURI 从 S3 下载文件,然后将其保存在本地,以便我可以使用 ActionMailer 将文件作为附件发送。

一些奇怪的事情正在发生。正在下载和附加的图像已损坏,图像的底部丢失。

这是代码:

require 'open-uri'
open("#{Rails.root.to_s}/tmp/#{a.attachment_file_name}", "wb") do |file|  
  source_url = a.authenticated_url()
  io = open(URI.parse(source_url).to_s)
  file << io.read
  attachments[a.attachment_file_name] = File.read("#{Rails.root.to_s}/tmp/#{a.attachment_file_name}")        
end

a是来自 ActionMailer 的附件。

接下来我可以尝试什么?

4

1 回答 1

9

看起来您正试图在文件关闭之前读取文件,这可能会使部分文件缓冲区未写入。

我会这样做:

require 'open-uri'

source_url = a.authenticated_url()
attachment_file = "#{Rails.root.to_s}/tmp/#{a.attachment_file_name}"
open(attachment_file, "wb") do |file|  
  file.print open(source_url, &:read)
end

attachments[a.attachment_file_name] = File.read(attachment_file)

它看起来source_url = a.authenticated_url()将是一个字符串,因此将字符串解析为 URI 然后to_s对其进行处理将是多余的,除非 URI 正在执行一些规范化,我认为它不会这样做。

根据我的系统管理员经验:一项附带任务是清理下载/假脱机文件。它们可以在附加后立即删除,或者您可以有一个每天运行的 cron 作业,删除一天前的所有假脱机文件。

另一个问题是在无法读取 URL 的情况下没有错误处理,导致附件失败。使用临时假脱机文件,您可以检查文件是否存在。更好的是,如果服务器返回 400 或 500 错误,您可能应该准备好处理异常。


为避免使用临时假脱机文件,请尝试以下未经测试的代码:

require 'open-uri'

source_url = a.authenticated_url()
attachments[a.attachment_file_name] = open(source_url, &:read)
于 2010-12-06T00:37:57.627 回答