Edgewall Software
Modify ↓

Opened 8 years ago

Closed 8 years ago

#12764 closed defect (fixed)

Subscribers without RecipientMatcher and from header in mail ignore use_short_addr and smtp_default_domain options

Reported by: Jun Omae Owned by: Jun Omae
Priority: normal Milestone: 1.2.2
Component: notification Version: 1.2
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Fix [notification] use_short_addr and smtp_default_domain options being ignored for subscriptions without email address.

API Changes:
Internal Changes:

Description

Currently, RecipientMatcher class handles use_short_addr and smtp_default_domain options in [notification]. It means subscriptions from subscribers without RecipientMatcher ignore their options.

For example, NewTicketSubscriber in CookBook/Notification/Subscriptions@11.

I consider the handling should move to EmailDistributor.distribute().

Attachments (0)

Change History (3)

comment:1 by Jun Omae, 8 years ago

Proposed changes in jomae.git@t12764 but it is unable to remove RecipientMatcher.match_recipients from existent ticket subscribers because it needs to solve authenticated from values in ticket properties (reporter, owner, cc).

comment:2 by Jun Omae, 8 years ago

I rebased and refactored the branch. I'm going to push to 1.2-stable if no objections.

comment:3 by Jun Omae, 8 years ago

Release Notes: modified (diff)
Resolution: → fixed
Status: assigned → closed

Committed in [15849] and merged in [15850].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jun Omae.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jun Omae to the specified user.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.