From ec1e98d3a29be264419372426c69b8c408971918 Mon Sep 17 00:00:00 2001 From: Tomas Kopecek Date: Dec 03 2024 11:20:54 +0000 Subject: Replace iter_content with copyfileobj It is a bit faster. With a local server it gives 70% of original time. For real network the gain would be smaller, but doesn't hurt to have it there. --- diff --git a/koji/__init__.py b/koji/__init__.py index 39ee74c..00a22d8 100644 --- a/koji/__init__.py +++ b/koji/__init__.py @@ -38,6 +38,7 @@ import os.path import pwd import random import re +import shutil import socket import struct import sys @@ -2028,8 +2029,7 @@ def downloadFile(url, path=None, fo=None): resp = request_with_retry().get(url, stream=True) try: - for chunk in resp.iter_content(chunk_size=8192): - fo.write(chunk) + shutil.copyfileobj(resp.raw, fo) finally: resp.close() resp.raise_for_status()