vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: handle None session.data in maintenance mode (fixes #33258)

Browse source 
- Add null check for session.data in safe_exec.py
- Add null check for session.data in base_template_page.py
- Prevents AttributeError when session.data is None during maintenance mode
This commit is contained in:
Venkatesh M 2025-07-19 16:02:57 +00:00
parent 996843f11b
commit 96e0f49df4
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/utils/safe_exec.py
View file

@ -232,13 +232,13 @@ def get_safe_globals():
get_fullname=frappe.utils.get_fullname,
get_gravatar=frappe.utils.get_gravatar_url,
full_name=frappe.local.session.data.full_name
if getattr(frappe.local, "session", None)
if getattr(frappe.local, "session", None) and getattr(frappe.local.session, "data", None)
else "Guest",
request=getattr(frappe.local, "request", {}),
session=frappe._dict(
user=user,
csrf_token=frappe.local.session.data.csrf_token
if getattr(frappe.local, "session", None)
if getattr(frappe.local, "session", None) and getattr(frappe.local.session, "data", None)
else "",
),
make_get_request=frappe.integrations.utils.make_get_request,

2
frappe/website/page_renderers/base_template_page.py
View file

@ -16,7 +16,7 @@ class BaseTemplatePage(BaseRenderer):
self.context.update(frappe.local.conf.get("website_context") or {})
def add_csrf_token(self, html):
if frappe.local.session:
if frappe.local.session and getattr(frappe.local.session, "data", None):
csrf_token = frappe.local.session.data.csrf_token
return html.replace(
"<!-- csrf_token -->", f'<script>frappe.csrf_token = "{csrf_token}";</script>'