Make sure we log failures in ensure block.

This commit is contained in:
Guo Xiang Tan 2016-07-15 11:36:47 +08:00
parent 4aa3442824
commit b981041f6f

View file

@ -52,12 +52,18 @@ module BackupRestore
rescue Exception => ex rescue Exception => ex
log "EXCEPTION: " + ex.message log "EXCEPTION: " + ex.message
log ex.backtrace.join("\n") log ex.backtrace.join("\n")
@success = false
else else
@success = true @success = true
"#{@archive_basename}.tar.gz" "#{@archive_basename}.tar.gz"
ensure ensure
notify_user rescue nil begin
remove_old rescue nil notify_user
remove_old
rescue => ex
Rails.logger.error("#{ex}\n" + ex.backtrace.join("\n"))
end
clean_up clean_up
@success ? log("[SUCCESS]") : log("[FAILED]") @success ? log("[SUCCESS]") : log("[FAILED]")
end end