Tempfileで作成したPDFファイルがスクリプト実行中に削除されてしまう

Question

1行のCSVデータでは問題なくzipファイルをモデルに保存できますが、複数行のCSVデータでは
No such file or directoryとなり、zipファイルをモデルに保存することができませんでした。

tmp/以下のディレクトリを実行中に見ると、勝手にPDFファイルが削除されているようでした。
以下、コードになります。ご教授いただけますと幸いです。

class SendMaskInsuranceFile < ApplicationRecord
  mount_uploader :csv_url, DocumentFileUploader
  mount_uploader :zip_url, ZipFileUploader

  validates :csv_url, presence: true

  def self.generate_mask_insurance_zip_file
    ActiveRecord::Base.transaction do
      SendMaskInsuranceFile.where(created_at: Time.current.yesterday..Time.current).each do |send_mask_insurance_file|
        Tempfile.create(["#{Time.current.to_i}", ".zip"], Rails.root.join('tmp')) do |zip_tempfile|
          Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|
            URI.open(send_mask_insurance_file.csv_url.path) do |f|
              csv = CSV.new(f, headers: :first_row)
              csv.each do |row|
                staff_master    = StaffMaster.find_by(id: row[5])
                employment_file = staff_master.original_insurance_files.where(file_type: :social).order(id: :desc).first
                social_file     = staff_master.original_insurance_files.where(file_type: :employment).order(id: :desc).first
                if staff_master
                  pdf_string = WickedPdf.new.pdf_from_string(
                    ActionController::Base.new().render_to_string(
                      template: '/admins/send_mask_insurance_files/sended.pdf.slim',
                      layout:   'layouts/pdf.pdf.slim',
                      encoding: 'utf-8',
                      locals:   {
                        postal_code:     row[3],
                        address:         row[4],
                        company_name:    row[2],
                        department_name: row[1],
                        employment_file: employment_file.blank? ? nil : embed_remote_image(employment_file.url.path, 'image/jpeg'),
                        social_file:     social_file.blank? ? nil : embed_remote_image(social_file.url.path, 'image/jpeg'),
                      }
                    )
                  ).force_encoding("UTF-8")

                  Tempfile.create(["#{staff_master.id}-#{Time.current.to_i}", ".pdf"], Rails.root.join('tmp')) do |mask_insurance_tempfile|
                    mask_insurance_tempfile.write pdf_string
                    zipfile.add(File.basename(mask_insurance_tempfile.path), mask_insurance_tempfile.path)
                  end
                end
              end
            end
          end
          send_mask_insurance_file.update!(zip_url: zip_tempfile)
        end
      end
    end
  end

  def self.embed_remote_image(url, content_type)
    asset  = URI.open(url, "r:UTF-8", &:read)
    base64 = Base64.encode64(asset.to_s).gsub(/\s+/, "")
    "data:#{content_type};base64,#{Rack::Utils.escape(base64)}"
  end
end

実行コマンド

bundle exec rake cron:generate_mask_insurance_zip_file

エラーメッセージ

[START] [{:generate_mask_insurance_zip_file=>:environment}] (2022/09/26 09:28)
[ERROR] No such file or directory @ rb_sysopen - /Users/●●/Dropbox/sites/app/tmp/02393164-166415211720220926-1793-w6nj56.pdf (2022/09/26 09:28)
rake aborted!
Errno::ENOENT: No such file or directory @ rb_sysopen - /Users/●●/Dropbox/sites/app/tmp/02393164-166415211720220926-1793-w6nj56.pdf
/Users/●●/Dropbox/sites/app/app/models/send_mask_insurance_file.rb:11:in `block (3 levels) in generate_mask_insurance_zip_file'
/Users/●●/Dropbox/sites/app/app/models/send_mask_insurance_file.rb:10:in `block (2 levels) in generate_mask_insurance_zip_file'
/Users/●●/Dropbox/sites/app/app/models/send_mask_insurance_file.rb:9:in `block in generate_mask_insurance_zip_file'
/Users/●●/Dropbox/sites/app/app/models/send_mask_insurance_file.rb:8:in `generate_mask_insurance_zip_file'
/Users/●●/Dropbox/sites/app/lib/tasks/cron.rake:98:in `block (2 levels) in <main>'
/Users/●●/Dropbox/sites/app/lib/tasks/cron.rake:7:in `block in task_with_logger_and_notify'
/Users/●●/.rbenv/versions/2.7.6/bin/bundle:23:in `load'
/Users/●●/.rbenv/versions/2.7.6/bin/bundle:23:in `<main>'
Tasks: TOP => cron:generate_mask_insurance_zip_file
(See full trace by running task with --trace)

Comments

[winterboum]

エラーメッセージは全文載せてください

[ssk]

ありがとうございます。
エラーメッセージを追加いたしました。

[winterboum]

send_mask_insurance_file.rb:10 って
zip_tempfile = Tempfile.new(["#{Time.current.to_i}", ".zip"], Rails.root.join('tmp'))
ですか？

[ssk]

10行目は以下になります。

Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|

[winterboum]

とすると、エラーメッセージが出たのは載せてある class SendMaskInsuranceFile とは別物ですね。
8行目が generate_mask_insurance_zip_file
9行目が block in generate_mask_insurance_zip_file
だと言ってますから
載せてるfileだとすると 10行目は SendMaskInsuranceFile.where(created_at:
になりますが。

[ssk]

申し訳ございません。
余計な改行が入っていました。

正しくは11行目が以下になります。
Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|

内容も修正いたしました。

Answer 1

Tempfile の一時ファイルは自動的に削除されます。
closeでtrueを指定しなかった場合でもGCで削除されます。
https://docs.ruby-lang.org/ja/latest/class/Tempfile.html#I_CLOSE

そのcsvを読むループの中でGCが動いているのかどうかはわかりませんが、
Tempfileの使い方としてはよくない（基本的に close の前に使う）と思います。

Comments

[ssk]

ありがとうございます。

Tempfile.createに変更することでブロックを抜けたタイミングで一時ファイルが削除されるかと思いますが、
Tempfile.createにしてもブロックを抜ける前に一時ファイルが削除されているようでした。
（エラーメッセージにも変化がありましたので、上記の内容を更新いたしました。）

[sigsegv]

zipfile.add()はzipに格納するファイルを登録するだけで、そのファイルをzipに書き出すのは、
Zip::File を close するとき（=openのブロックを抜けるとき）だと思います。
つまり、zipfile.add() の時に一時ファイルが存在しているだけでは不十分で、
Zip::File.open ～ を抜けるときまで一時ファイルが残っていないとだめです。

[winterboum]

初見はそこだったのですが、この場合
「zipfile.add の行でエラー」
とならないのでしょうか？
Zip::File.open の loopの中ではありますが。

[sigsegv]

zipfile.add の時点は、ファイル（一時ファイル）が存在するのでエラーになりません。

[sigsegv]

なんか伝わってない気がしますね。

zipfile.add() ではそのファイルが存在するかどうか、通常ファイルかどうかなどをチェックして、
問題なければ zipfileオブジェクト内部のリストにそのパスを追加します。
最後に(zipfileをクローズするとき）にリストにあるパスを順次読み出し、zipファイルを作成します。
この時点でファイルが削除されているとエラーとなります。これが質問のエラーです。

細部は違ってるかもしれませんが、概ね上記のような動きをしているのだろうというのが回答内容です。

Answer 2

なかなか不思議な現象です。
１０行目で
zip_tempfile = Tempfile.new(["#{Time.current.to_i}", ".zip"], Rails.root.join('tmp'))
と、zip_tempfile の拡張子は zip である、としているのに 11行目の
Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|
で起きたエラーでは
/Users/●●/Dropbox/sites/app/tmp/02393164-166410115520220925-6791-ba6on1.pdf
pdf だと言ってる。
次の行ですから値が変わるはずがないのに。
これを見ると載っている class SendMaskInsuranceFile が吐いたエラーに見えないのですが。

追記
ついでに書いておきますが
この問題が解決し

Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|
end
 send_mask_insurance_file.update!(zip_url: zip_tempfile)

まで無事に終わっても、send_mask_insurance_file.zip_url が示す先にはfileの実体はありませんよ。
Tempfileなので削除されます。

Comments

[ssk]

ありがとうございます。
コードのTempfile.newをTempfile.createに変更し、ブロックで処理してみましたが。
同じようにpdfを保持？することができませんでした....
（エラーメッセージにも変化がありましたので、上記の内容を更新いたしました。