Edgewall Software
Modify

Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#3057 closed enhancement (fixed)

New smtp field on config

Reported by: Pedro Algarvio, aka, s0undt3ch <ufs@…> Owned by: Emmanuel Blot
Priority: high Milestone:
Component: ticket system Version: devel
Severity: normal Keywords: notification email
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

There's a need to introduce yet another field in trac.ini

Some mailing lists only accept mail if it's email address is on Cc or To.

Now, by default trac sends emails as BCc. Normally this will be fine cuz mailing list addresses are public, but what if a user also wants a copy to a private email address? Theres no way trac can handle this, unless we add smtp_always_bcc to smtp_always_cc, and probably we can remove allow_public_cc or leave it for compatability reasons.

Either way, trac should be able to address this issue.

See also #2861.

Attachments (1)

bcc.patch (5.2 KB ) - added by Emmanuel Blot 19 years ago.
Proposed patch - not complete, needs unit test(s)

Download all attachments as: .zip

Change History (6)

comment:1 by Christopher Lenz, 19 years ago

Version: 0.9.5devel

We've experienced this problem when upgrading this very site to 0.10dev.

Emmanuel?

comment:2 by Emmanuel Blot, 19 years ago

I fully agree we should move to explicit smtp_always_cc and smtp_always_bcc config option.

I've introduced the infamous allow_public_cc to avoid breaking the compatibility with the existing configuration, but I eventually find this choice a very bad idea.

If Trac developers are ok with removing this option and using explicit cc/bcc options, I can work on this change.

comment:3 by Christopher Lenz, 19 years ago

+1

comment:4 by Emmanuel Blot, 19 years ago

Owner: changed from Jonas Borgström to Emmanuel Blot
Priority: normalhigh
Status: newassigned

To sum up the conversation we had on IRC:

  • create a new config option stmp_always_bcc
  • rename allow_public_cc option with use_public_cc
    • this option will only be applied to the ticket CC: , reporter and owner fields, not to the smtp_always_* lists

→ mailing list address needs to be declared in the smtp_always_cc list.

by Emmanuel Blot, 19 years ago

Attachment: bcc.patch added

Proposed patch - not complete, needs unit test(s)

comment:5 by Emmanuel Blot, 19 years ago

Resolution: fixed
Status: assignedclosed

Should be fixed in r3231

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.