DGNum/demarches-normaliennes
13
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #10247 from colinux/fix-export-memory-leak

Browse source 
Tech: fix memory leak pendant le téléchargement des PJ des exports
This commit is contained in:
mfo 2024-04-02 08:14:28 +00:00 • committed by GitHub
commit e40ef51f61
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
app/lib/download_manager/parallel_download_queue.rb
View file

@ -14,7 +14,7 @@ module DownloadManager
def download_all def download_all
hydra = Typhoeus::Hydra.new(max_concurrency: DOWNLOAD_MAX_PARALLEL) hydra = Typhoeus::Hydra.new(max_concurrency: DOWNLOAD_MAX_PARALLEL)
attachments.map do |attachment, path| attachments.each do |attachment, path|
begin begin
download_one(attachment: attachment, download_one(attachment: attachment,
path_in_download_dir: path, path_in_download_dir: path,
@ -23,7 +23,10 @@ module DownloadManager
on_error.call(attachment, path, e) on_error.call(attachment, path, e)
end end
end end
hydra.run hydra.run
GC.start
end end
# can't be used with typhoeus, otherwise block is closed before the request is run by hydra # can't be used with typhoeus, otherwise block is closed before the request is run by hydra