From e20ae955028bd2fd4730e01b9b461461693fa790 Mon Sep 17 00:00:00 2001 From: Jan Friedli Date: Thu, 21 May 2020 18:09:38 +0200 Subject: [PATCH] removed randint from bandit inspection --- matweb/file_removal_scheduler.py | 21 ++++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/matweb/file_removal_scheduler.py b/matweb/file_removal_scheduler.py index 59bb9ca..d0f0d4b 100644 --- a/matweb/file_removal_scheduler.py +++ b/matweb/file_removal_scheduler.py @@ -1,3 +1,8 @@ +""" +Check regularly if files have not been downloaded +and delete them if they're too old +""" + import glob import time import sys @@ -7,13 +12,23 @@ import random from matweb import utils -def run_file_removal_job(upload_folder_path): - if random.randint(0, 10) == 0: +def run_file_removal_job(upload_folder_path: str): + """ + Check for removals at about every 10th request + :param upload_folder_path: path were the uploaded files are stored + :return: Void + """ + if random.randint(0, 10) == 0: # nosec for file in glob.glob(upload_folder_path + '/*'): delete_file_when_too_old(file) -def delete_file_when_too_old(filepath): +def delete_file_when_too_old(filepath: str): + """ + Check if files are too old and remove em + :param filepath: Path of the file to check + :return: Void + """ file_mod_time = os.stat(filepath).st_mtime # time in second since last modification of file