Removed checks from 'validate_and_sanitise' functions for whether a Path object is a file

Author: tieneupin

src/murfey/server/api/clem.py

@@ -94,17 +94,6 @@ def validate_and_sanitise(
     if not str(full_path).startswith(str(base_path)):
         raise ValueError(f"{file} points to a directory that is not permitted")
 
-    # # Check that it's a file, not a directory
-    # # Make a couple of attempts to rule out race condition
-    # attempts = 0
-    # while attempts < 50:
-    #     if full_path.is_file() is True:
-    #         break
-    #     attempts += 1
-    #     time.sleep(0.1)
-    # else:
-    #     raise ValueError(f"{file} is not a file")
-
     # Check that it is of a permitted file type
     if f"{full_path.suffix}" not in valid_file_types:
         raise ValueError(f"{full_path.suffix} is not a permitted file format")

src/murfey/workflows/clem/__init__.py

@@ -2,7 +2,6 @@
 
 import logging
 import re
-import time
 from pathlib import Path
 from typing import Optional, Type, Union
 
@@ -90,17 +89,6 @@ def _validate_and_sanitise(
     if not str(full_path).startswith(str(base_path)):
         raise ValueError(f"{file} points to a directory that is not permitted")
 
-    # Check that it's a file, not a directory
-    # Make a couple of attempts to rule out race condition
-    attempts = 0
-    while attempts < 50:
-        if full_path.is_file() is True:
-            break
-        attempts += 1
-        time.sleep(0.1)
-    else:
-        raise ValueError(f"{file} is not a file")
-
     # Check that it is of a permitted file type
     if f"{full_path.suffix}" not in valid_file_types:
         raise ValueError(f"{full_path.suffix} is not a permitted file format")