Modify ↓
Opened 12 years ago
Last modified 8 years ago
#10676 new defect
StaticEmailDecorator doesn't modify recipient list (RCPT TO)
Reported by: | anonymous | Owned by: | |
---|---|---|---|
Priority: | normal | Component: | AnnouncerPlugin |
Severity: | normal | Keywords: | |
Cc: | xaj | Trac Release: | 1.0 |
Description (last modified by )
The StaticEmailDecorator only modifies the message body headers but doesn't result in the actual recipient list used in the SMTP RCPT TO command being extended.
Attachments (0)
Change History (6)
comment:1 Changed 12 years ago by
Cc: | xaj added; anonymous removed |
---|
comment:2 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 12 years ago by
comment:4 Changed 12 years ago by
#8306 hints on checking/correctly handling of duplicated addresses.
comment:6 Changed 8 years ago by
Owner: | Steffen Hoffmann deleted |
---|
Note: See
TracTickets for help on using
tickets.
As a matter of fact in current code the recipients selection is final before decorators get involved.
So the aforementioned
[announcer]
optionsemail_always_cc
andemail_always_bcc
can't work. Period. Following the decorator approach we must decorate before evaluating and filtering recipients list.Evaluating these options shouldn't be done in a decorator at all. Reasonable: It must be done near recipient/subscription filtering anyway. On more step forward: My immediate feeling is, that this is just another subscriber, but I'll have to look at it more closely for a finally educated decision.
Final note for now, that this issue is somewhat linked to the issue of processing message with encryption (see r8087) or 'private' messages (i.e. these coming from AccountManagerPlugin, see related #8796) right as well.