Edgewall Software
Modify

Opened 13 years ago

Last modified 7 months ago

#10112 new defect

notifications strip slashes from email addresses

Reported by: bpkroth@… Owned by:
Priority: normal Milestone: next-stable-1.6.x
Component: notification Version: 0.11.7
Severity: normal Keywords:
Cc: Ryan J Ollos Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Ryan J Ollos)

We setup aliases in sendmail like group/mytestgroup@domain.com populated with the members of mytestgroup for easy/simplistic list aliases. The left hand side of these appears to be getting stripped when sending email through trac's smtp_notification features so that the logs show it only sending messages to mytestgroup@domain.com which typically isn't a valid alias.

I'm running trac from Debian Squeeze - 0.11.7.

Let me know if you need anything else.

Thanks, Brian

Attachments (0)

Change History (6)

comment:1 by Remy Blank, 13 years ago

Milestone: next-minor-0.12.x

Right, we're a bit too restrictive here. Wikipedia says that the local-part of the e-mail address may use any of the following ASCII characters:

  • Uppercase and lowercase English letters (a–z, A–Z)
  • Digits 0 to 9
  • Characters ! # $ % & ' * + - / = ? ^ _ ` { | } ~
  • Character . (dot, period, full stop) provided that it is not the first or last character, and provided also that it does not appear two or more times consecutively

RFC5322, section 3.4.1 seems to be the authoritative source. We won't add the = back (see #9898), but the others are candidates.

comment:2 by Ryan J Ollos, 10 years ago

Cc: Ryan J Ollos added
Description: modified (diff)

comment:3 by Ryan J Ollos, 9 years ago

Milestone: next-minor-0.12.xnext-stable-1.0.x

comment:4 by Ryan J Ollos, 7 years ago

Milestone: next-stable-1.0.xnext-stable-1.2.x

Moved ticket assigned to next-stable-1.0.x since maintenance of 1.0.x is coming to a close. Please move the ticket back if it's critical to fix on 1.0.x.

comment:5 by Ryan J Ollos, 4 years ago

Milestone: next-stable-1.2.xnext-stable-1.4.x

comment:6 by Ryan J Ollos, 7 months ago

Milestone: next-stable-1.4.xnext-stable-1.6.x

Milestone renamed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


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