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 Ryan J Ollos)

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 Russell Jackson

Cc: xaj added; anonymous removed

comment:2 Changed 12 years ago by Ryan J Ollos

Description: modified (diff)

comment:3 Changed 12 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 12 years ago by Steffen Hoffmann

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

comment:5 Changed 12 years ago by Steffen Hoffmann

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

comment:6 Changed 8 years ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.