Merge pull request #11257 from scmmishra/same-site-mobile
feat: set samesite none for mobile
This commit is contained in:
commit
fbb3d6b88e
1 changed files with 5 additions and 0 deletions
|
|
@ -340,6 +340,11 @@ class CookieManager:
|
|||
def set_cookie(self, key, value, expires=None, secure=False, httponly=False, samesite="Lax"):
|
||||
if not secure and hasattr(frappe.local, 'request'):
|
||||
secure = frappe.local.request.scheme == "https"
|
||||
|
||||
# Cordova does not work with Lax
|
||||
if frappe.local.session.data.device == "mobile":
|
||||
samesite = None
|
||||
|
||||
self.cookies[key] = {
|
||||
"value": value,
|
||||
"expires": expires,
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue