Mercurial > moin > 1.9

changeset 4358:a952d07dea69

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
TextChas: fix treatment of unknown keys
author Thomas Waldmann <tw AT waldmann-edv DOT de>
date Tue, 30 Sep 2008 09:32:53 +0200
parents 1f7b0a55955b
children 0bf173b2d64a
files MoinMoin/security/textcha.py
diffstat 1 files changed, 7 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/MoinMoin/security/textcha.py	Mon Sep 29 21:32:16 2008 +0200
+++ b/MoinMoin/security/textcha.py	Tue Sep 30 09:32:53 2008 +0200
@@ -89,8 +89,8 @@
                 self.answer_re = re.compile(self.answer_regex, re.U|re.I)
             except KeyError:
                 # this question does not exist, thus there is no answer
-                self.answer_regex = ur"[^.]*" # this shall never match!
-                self.answer_re = re.compile(self.answer_regex, re.U|re.I)
+                self.answer_regex = ur"[Never match for cheaters]"
+                self.answer_re = None
                 logging.warning(u"TextCha: Non-existing question '%s'. User '%s' trying to cheat?" % (
                                 self.question, self.user_info))
             except re.error:
@@ -116,7 +116,11 @@
     def check_answer(self, given_answer):
         """ check if the given answer to the question is correct """
         if self.is_enabled():
-            success = self.answer_re.match(given_answer.strip()) is not None
+            if self.answer_re is not None:
+                success = self.answer_re.match(given_answer.strip()) is not None
+            else:
+                # someone trying to cheat!?
+                success = False
             success_status = success and u"success" or u"failure"
             logging.info(u"TextCha: %s (u='%s', a='%s', re='%s', q='%s')" % (
                              success_status,