Edgewall Software
Modify

Ticket #2434 (closed enhancement: fixed)

Opened 6 years ago

Last modified 5 years ago

Separate emails sent for each recipient

Reported by: Alex Buccino <alexander.buccino@…> Owned by: eblot
Priority: normal Milestone: 0.10
Component: ticket system Version: 0.9
Severity: normal Keywords: notification email
Cc:
Release Notes:
API Changes:

Description

Trac sends out separate emails for each recipient which meant that putting email aliases in the CC field could mean people would get duplicate messages. If all the addresses were placed in the To and Cc fields then email servers could weed out duplicates in the standard way.

Attachments

Change History

comment:1 Changed 6 years ago by eblot

  • Owner changed from jonas to eblot
  • Status changed from new to assigned

Note: sandbox/notification [2598] implements this feature

comment:2 Changed 6 years ago by Alex Buccino <alexander.buccino@…>

Just a note: While this is as much as Trac can do, it may not address the issue of multiple emails going to users that are both addressed directly and are members of an alias. Some mail systems may handle this gracefully, but I now realize that at least our email server, Lotus Notes, is only able to weed out duplicates sent from within Notes. It appears that outside emails are always sent to all recipients, even if they are already receiving a copy as members of an alias.

comment:3 Changed 6 years ago by Gunnar Wagenknecht <gunnar@…>

  • Cc gunnar@… added

It would be better to collect all mail addresses for notification and remove duplicates from that list before sending notifications.

I prefere one mail per recipient and not putting everybody on CC/TO. Please not that this is also problematic from a data privacy point of view in some countries (eg. Germany). Mailinglisst or systems sending out mail to multiple recipients must ensure that recipients can't see who got this mail too.

comment:4 Changed 6 years ago by Gunnar Wagenknecht <gunnar@…>

Paragraph two line one: s/Please not/Please note
;)

comment:5 Changed 6 years ago by eblot

  • Keywords notification email added

This issue is addressed up to some level in the notification sandbox.

There is a configuration option to either send the CC list as Cc: or Bcc: (to fulfill the privacy requirement). Duplicates are also removed (as long as they appear with the exact same syntax, no alias resolution is done here) before emails are sent.

I do not think one mail / recipient is a good idea, as notifications are processed synchronoulsy within the web request: the web server sends the next page once the communication with the SMTP server is complete. The current implementation uses one SMTP connection per recipient. Although this might be optimized, I think it is preferable to send one email for all recipients - there may be a flaw here if the list of recipients is very long, but I think this could be addressed in a second step.

comment:6 Changed 6 years ago by Gunnar Wagenknecht <gunnar@…>

There is a configuration option to either send the CC list as Cc: or Bcc: (to fulfill the privacy requirement)

What's contained in the To: header?

comment:7 Changed 6 years ago by eblot

To: reporter, owner (as long as I remember)
To: is always visible whatever the Cc/Bcc setting.

comment:8 Changed 6 years ago by eblot

  • Milestone set to 0.10
  • Resolution set to fixed
  • Status changed from assigned to closed

Should be fixed in [2799]

comment:9 Changed 6 years ago by eblot

See also #2727.

comment:10 Changed 6 years ago by anonymous

  • Cc gunnar@… removed
View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from eblot. Next status will be 'closed'
Author


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

 
Note: See TracTickets for help on using tickets.