Edgewall Software

Ticket #3057 (closed enhancement: fixed)

Opened 3 years ago

Last modified 2 years ago

New smtp field on config

Reported by: Pedro Algarvio, aka, s0undt3ch <ufs@…> Owned by: eblot
Priority: high Milestone:
Component: ticket system Version: devel
Severity: normal Keywords: notification email
Cc:

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

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

Change History

Changed 3 years ago by cmlenz

  • version changed from 0.9.5 to devel

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

Emmanuel?

Changed 3 years ago by eblot

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.

Changed 3 years ago by cmlenz

+1

Changed 3 years ago by eblot

  • owner changed from jonas to eblot
  • priority changed from normal to high
  • status changed from new to assigned

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.

Changed 3 years ago by eblot

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

Changed 3 years ago by eblot

  • status changed from assigned to closed
  • resolution set to fixed

Should be fixed in r3231

Add/Change #3057 (New smtp field on config)

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.