summaryrefslogtreecommitdiffstats
path: root/core/utils.py
diff options
context:
space:
mode:
Diffstat (limited to 'core/utils.py')
-rw-r--r--core/utils.py17
1 files changed, 13 insertions, 4 deletions
diff --git a/core/utils.py b/core/utils.py
index 8e8d6e4..a2f55dd 100644
--- a/core/utils.py
+++ b/core/utils.py
@@ -79,10 +79,14 @@ except:
pass
def get_cookie_max_age():
- """This is where we detect whether the machine the user is using is a shared-use device or a personbal device.
+ """This is where we detect whether the machine the user is using is a shared-use device or a personal device.
If it is shared-use, then we set a much shorter cookie timout period.
"""
- return COOKIE_MAX_AGE
+ if shared_use_machine():
+ return COOKIE_SHORT_TIMEOUT
+ else:
+ return COOKIE_MAX_AGE
+
def sanitize_name(name):
"""Filenames sould not contain these characters as then the system barf when it tries to use them in URLs
@@ -304,10 +308,15 @@ def get_git_string(user):
return None
person = people[0]
return f"{person.fullname} <{user.email}>"
-
+
+def shared_use_machine():
+ """Looks for a cookie which only exists on shared use machines
+ """
+ return False
+
def get_cookie(request):
"""The initial idea of having a default turned out to be a bad idea as people just ignore the field.
- if the default value is blank, then the form validation code makes the user type something in.
+ If the default value is blank, then the form validation code makes the user type something in.
So having a blank is best if the cookie is to be displayed as the first value seen on a form.
But if the cookie is to be stored, then "Unset" may be better.
"""