FIX: warn instead of excepting if MaxMind DBs can't be downloaded (#7600)

This commit is contained in:
Michael Brown 2019-05-24 18:08:00 -04:00 committed by Régis Hanol
parent 3bc99e5372
commit 5bfeef7705

View File

@ -27,16 +27,20 @@ class DiscourseIpInfo
def self.mmdb_download(name)
FileUtils.mkdir_p(path)
gz_file = FileHelper.download(
"https://geolite.maxmind.com/geoip/databases/#{name}/update",
max_file_size: 100.megabytes,
tmp_file_name: "#{name}.gz"
)
begin
gz_file = FileHelper.download(
"https://geolite.maxmind.com/geoip/databases/#{name}/update",
max_file_size: 100.megabytes,
tmp_file_name: "#{name}.gz"
)
Discourse::Utils.execute_command("gunzip", gz_file.path)
Discourse::Utils.execute_command("gunzip", gz_file.path)
path = gz_file.path.sub(/\.gz\z/, "")
FileUtils.mv(path, mmdb_path(name))
path = gz_file.path.sub(/\.gz\z/, "")
FileUtils.mv(path, mmdb_path(name))
rescue HTTPError => e
Rails.logger.warn("MaxMindDB (#{name}) could not be downloaded: #{e}")
end
ensure
gz_file&.close!
end