From 54cf0362ca672577da49a73fe2479c2cfb67b8aa Mon Sep 17 00:00:00 2001 From: Saifeddine ALOUI Date: Sun, 10 Mar 2024 01:21:12 +0100 Subject: [PATCH] fix in security --- lollms/security.py | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/lollms/security.py b/lollms/security.py index 70495e7..39cd5ef 100644 --- a/lollms/security.py +++ b/lollms/security.py @@ -3,6 +3,7 @@ from ascii_colors import ASCIIColors from pathlib import Path from typing import List import os +import re def sanitize_path(path:str, allow_absolute_path:bool=False, error_text="Absolute database path detected", exception_text="Detected an attempt of path traversal. Are you kidding me?"): if path is None: @@ -23,6 +24,20 @@ def sanitize_path_from_endpoint(path:str, error_text="A suspected LFI attack det raise HTTPException(status_code=400, detail=exception_text) return path + +def sanitize_path_from_endpoint(path: str, error_text="A suspected LFI attack detected. The path sent to the server has suspicious elements in it!", exception_text="Invalid path!"): + if path is None: + return path + + # Regular expression to detect patterns like "...." and multiple forward slashes + suspicious_patterns = re.compile(r'(\.\.+)|(/+/)') + + if suspicious_patterns.search(path) or Path(path).is_absolute(): + ASCIIColors.error(error_text) + raise HTTPException(status_code=400, detail=exception_text) + + return path + def forbid_remote_access(lollmsElfServer): if lollmsElfServer.config.host!="localhost" and lollmsElfServer.config.host!="127.0.0.1": raise Exception("This functionality is forbidden if the server is exposed") @@ -42,3 +57,11 @@ def validate_path(path, allowed_paths:List[str|Path]): # If the path is not within any of the allowed paths, return False return False + + +if __name__=="__main__": + sanitize_path_from_endpoint("main") + sanitize_path_from_endpoint("cat/main") + print("Main passed") + sanitize_path_from_endpoint(".../user") + print("hi") \ No newline at end of file