Opened 9 years ago

StaticEmailDecorator doesn't modify recipient list (RCPT TO)

Reported by: Owned by: anonymous normal AnnouncerPlugin normal xaj 1.0

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.

comment:2 Changed 9 years ago by Ryan J Ollos

Description: modified (diff)

comment:3 Changed 9 years ago by Steffen Hoffmann

As a matter of fact in current code the recipients selection is final before decorators get involved.

So the aforementioned [announcer] options email_always_cc and email_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.

comment:4 Changed 9 years ago by Steffen Hoffmann

#8306 hints on checking/correctly handling of duplicated addresses.

comment:5 Changed 9 years ago by Steffen Hoffmann

#11060 has been closed as a duplicate of this ticket.

comment:6 Changed 5 years ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

Modify Ticket

Change Properties