From 463a1d8c7c6a1deefc0ada8852c4f84afbdf61ad Mon Sep 17 00:00:00 2001 From: Jannat Patel Date: Fri, 6 Dec 2024 15:14:03 +0530 Subject: [PATCH] fix: check if its file before fetching --- lms/page_renderers.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/lms/page_renderers.py b/lms/page_renderers.py index 9b9ee96d..68daa5b6 100644 --- a/lms/page_renderers.py +++ b/lms/page_renderers.py @@ -154,9 +154,11 @@ class SCORMRenderer(BaseRenderer): if not extension: path = f"{path}.html" - f = open(path, "rb") - response = Response( - wrap_file(frappe.local.request.environ, f), direct_passthrough=True - ) - response.mimetype = mimetypes.guess_type(path)[0] - return response + # check if path exists and is actually a file and not a folder + if os.path.exists(path) and os.path.isfile(path): + f = open(path, "rb") + response = Response( + wrap_file(frappe.local.request.environ, f), direct_passthrough=True + ) + response.mimetype = mimetypes.guess_type(path)[0] + return response