Edgewall Software

Ticket #5789 (closed defect: duplicate)

Opened 18 months ago

Last modified 17 months ago

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

Reported by: anonymous Owned by: eblot
Priority: normal Milestone:
Component: notification Version: 0.10.4
Severity: normal Keywords:
Cc:

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

Change History

Changed 18 months ago by thatch

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.

Changed 17 months ago by eblot

  • status changed from new to closed
  • resolution set to duplicate

Duplicate of #4372

Add/Change #5789 (Trac's notification.py doesn't properly handle some email addresses)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from eblot. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.