Modify

Opened 2 years ago

Last modified 19 months ago

#10676 new defect

StaticEmailDecorator doesn't modify recipient list (RCPT TO)

Reported by: anonymous Owned by: hasienda
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords:
Cc: xaj Trac Release: 1.0

Description (last modified by rjollos)

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 (5)

comment:1 Changed 22 months ago by xraj

  • Cc xaj added; anonymous removed

comment:2 Changed 20 months ago by rjollos

  • Description modified (diff)

comment:3 Changed 19 months ago by hasienda

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 19 months ago by hasienda

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

comment:5 Changed 19 months ago by hasienda

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

Add Comment

Modify Ticket

Action
as new The owner will remain hasienda.
Author


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

 
Note: See TracTickets for help on using tickets.