aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat (limited to '')
-rwxr-xr-xMailman/Cgi/subscribe.py4
-rwxr-xr-xMailman/Defaults.py.in9
-rw-r--r--NEWS8
3 files changed, 16 insertions, 5 deletions
diff --git a/Mailman/Cgi/subscribe.py b/Mailman/Cgi/subscribe.py
index 0fde280a..d6b1517d 100755
--- a/Mailman/Cgi/subscribe.py
+++ b/Mailman/Cgi/subscribe.py
@@ -136,6 +136,10 @@ def process_form(mlist, doc, cgidata, lang):
remote).hexdigest()
if now - then > mm_cfg.FORM_LIFETIME:
results.append(_('The form is too old. Please GET it again.'))
+ if now - then < mm_cfg.SUBSCRIBE_FORM_MIN_TIME:
+ results.append(
+ _('Please take a few seconds to fill out the form before submitting it.')
+ )
if token != fhash:
results.append(_('You must GET the form before submitting it.'))
# Was an attempt made to subscribe the list to itself?
diff --git a/Mailman/Defaults.py.in b/Mailman/Defaults.py.in
index 2b0aa3ed..be1ac735 100755
--- a/Mailman/Defaults.py.in
+++ b/Mailman/Defaults.py.in
@@ -117,8 +117,8 @@ FORM_LIFETIME = hours(1)
# This hash is checked upon form submission and the subscribe fails if it
# doesn't match. I.e. the form posted must be first retrieved from the
# listinfo CGI by the same IP that posts it. The subscribe also fails if
-# the time the form was retrieved is more than the above FORM_LIFETIME
-# before submission.
+# the time the form was retrieved is more than the above FORM_LIFETIME or less
+# than the below SUBSCRIBE_FORM_MIN_TIME before submission.
# Important: If you have any static subscribe forms on your web site, setting
# this option will break them. With this option set, subscribe forms must be
# dynamically generated to include the hidden data. See the code block
@@ -126,6 +126,11 @@ FORM_LIFETIME = hours(1)
# for the details of the hidden data.
SUBSCRIBE_FORM_SECRET = None
+# If SUBSCRIBE_FORM_SECRET is not None, this is the minimum time the user must
+# take after retrieving the form before submitting it. Set to 0 to skip this
+# test.
+SUBSCRIBE_FORM_MIN_TIME = seconds(5)
+
# Command that is used to convert text/html parts into plain text. This
# should output results to standard output. %(filename)s will contain the
# name of the temporary file that the program should operate on.
diff --git a/NEWS b/NEWS
index e1bf51a0..45430743 100644
--- a/NEWS
+++ b/NEWS
@@ -13,9 +13,11 @@ Here is a history of user visible changes to Mailman.
a dynamically generated, hidden hash in the listinfo subscribe form and
check it upon submission. Setting this will prevent automated processes
(bots) from successfully POSTing web subscribes without first retrieving
- and parsing the form from the listinfo page. Note that enabling this
- will break ant static subscribe forms on your site. See the description
- in Defaults.py for more info. (LP: 1082746)
+ and parsing the form from the listinfo page. The form must also be
+ submitted no later than FORM_LIFETIME nor no earlier than
+ SUBSCRIBE_FORM_MIN_TIME after retrieval. Note that enabling this will
+ break any static subscribe forms on your site. See the description in
+ Defaults.py for more info. (LP: 1082746)
- add_members now has an option to add members with mail delivery disabled
by admin. (LP: 1070574)