Edgewall Software

Opened 13 years ago

Last modified 8 months ago

#10112 new defect

notifications strip slashes from email addresses — at Version 2

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

Change History (2)

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)
Note: See TracTickets for help on using tickets.