Skip to content

Commit 5fe7d41

Browse files
committed
fix: add timeouts to requests.get
1 parent 2e3ef37 commit 5fe7d41

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

imgsync/distros/__init__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ def _download_one(self, url, checksum):
116116
"""
117117
with tempfile.NamedTemporaryFile(suffix=".imgsync", delete=False) as location:
118118
try:
119-
response = requests.get(url, stream=True)
119+
response = requests.get(url, stream=True, timeout=10)
120120
except Exception as e:
121121
os.remove(location.name)
122122
LOG.error(e)

imgsync/distros/centos.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def __init__(self):
6767

6868
def _get_index(self):
6969
url = self.url + "image-index"
70-
r = requests.get(url)
70+
r = requests.get(url, timeout=10)
7171
if r.ok:
7272
aux = r.text
7373
else:

imgsync/distros/debian.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ def _sync_latest(self):
4343
base_url = self.url
4444

4545
checksum_file = base_url + "SHA512SUMS"
46-
checksum_file = requests.get(checksum_file)
46+
checksum_file = requests.get(checksum_file, timeout=10)
4747
if checksum_file.status_code != 200:
4848
LOG.error("Could not get checksums file %s" % checksum_file.url)
4949
return

imgsync/distros/ubuntu.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def _sync_latest(self):
4949
LOG.info("Downloading %s", filename)
5050
base_url = self.url + "current/"
5151
checksum_file = base_url + "SHA256SUMS"
52-
checksum_file = requests.get(checksum_file)
52+
checksum_file = requests.get(checksum_file, timeout=10)
5353
if checksum_file.status_code != 200:
5454
LOG.error("Could not get checksums file %s" % checksum_file.url)
5555
return

0 commit comments

Comments
 (0)