Edgewall Software
Modify

Opened 17 years ago

Closed 17 years ago

#5789 closed defect (duplicate)

Trac's notification.py doesn't properly handle some email addresses

Reported by: anonymous Owned by: Emmanuel Blot
Priority: normal Milestone:
Component: notification Version: 0.10.4
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I'm using trac on my lan and I have internal email addresses of the form user@…

In notification.py, addrfmt is used to extract the email address:

addrfmt = r"[\w\d_\.\-\+=]+\@(([\w\d\-])+\.)+([\w\d]{2,4})+"

and this regex expects to find a 2-to-4 \w\d ending, it won't match local, but only "loca".

This is why in my case, trac will try to send notification emails to user@…
The quickest hack in my case is to change the 4 to 5 :).

Attachments (0)

Change History (2)

comment:1 by Tim Hatch, 17 years ago

This is now an option in 0.11-dev. See r5516, which appears to apply okay to the 0.10 tree if you grab the diff.

comment:2 by Emmanuel Blot, 17 years ago

Resolution: duplicate
Status: newclosed

Duplicate of #4372

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Emmanuel Blot.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Emmanuel Blot to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.