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

perf: Don't query redirects on existing session

Browse source 
This is only used for a new user. After that it's never used but still
present in EVERY request!

https://ankush.dev/p/flamegraph-missing-forest-for-trees

towards https://github.com/frappe/caffeine/issues/15
This commit is contained in:
Ankush Menat 2025-01-01 14:29:36 +05:30
parent 171eab1387
commit cbb39d463a
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
frappe/auth.py
View file

@ -205,8 +205,7 @@ class LoginManager:
frappe.response["full_name"] = self.full_name frappe.response["full_name"] = self.full_name
# redirect information # redirect information
redirect_to = frappe.cache.hget("redirect_after_login", self.user) if not resume and (redirect_to := frappe.cache.hget("redirect_after_login", self.user)):
if redirect_to:
frappe.local.response["redirect_to"] = redirect_to frappe.local.response["redirect_to"] = redirect_to
frappe.cache.hdel("redirect_after_login", self.user) frappe.cache.hdel("redirect_after_login", self.user)