Ticket #3057 (closed enhancement: fixed)
Opened 6 years ago
Last modified 5 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: | |||
| Release Notes: | |||
| API 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
Change History
comment:1 Changed 6 years ago by cmlenz
- Version changed from 0.9.5 to devel
comment:2 Changed 6 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.
comment:3 Changed 6 years ago by cmlenz
+1
comment:4 Changed 6 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 6 years ago by eblot
Proposed patch - not complete, needs unit test(s)
comment:5 Changed 6 years ago by eblot
- Resolution set to fixed
- Status changed from assigned to closed
Should be fixed in r3231



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