Opened 8 years ago

Last modified 7 years ago

#7022 new defect

validate_author won't validate email address stored in trac preferences.

Reported by: andrew.beck@… Owned by: Richard Liao
Priority: normal Component: TracTicketValidatorPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11


When using the following config in trac.ini

[components] ticketvalidator.* = enabled

[ticketvalidator] validate_author = true validate_author.tip = Please fill your valid email address

and an anonymous user saves his email in trac preferences cookie e.g. http://hostname/projects/pathtoproject/prefs

then the Author field becomes something like "Andrew Beck <andrew.beck@…>" which fails validation where just andrew.beck@… works

Attachments (0)

Change History (1)

comment:1 Changed 7 years ago by lee.calabrese@…

This ticket is over 11 months old, but I just ran into the same problem, which I was able to fix relatively quickly. I don't believe this has any side-effects as I just changed the regex to also accept email addresses in the form "Name <email@…>".

--- 2010-01-05 03:35:32.000000000 -0800
+++  2011-03-06 21:34:17.000000000 -0800
@@ -89,7 +89,7 @@
             return False

         if len(email) > 7:
-            if re.match("^[a-zA-Z0-9._%-]+@[a-zA-Z0-9._%-]+\.[a-zA-Z]{2,6}$", email) != None:
+            if re.match("^(?P<named>.*? <){,1}[a-zA-Z0-9._%-]+@[a-zA-Z0-9._%-]+\.[a-zA-Z]{2,6}(?(named)>)$", email) != None:
                 return True
         return False

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain Richard Liao.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.