Edgewall Software
Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#12759 closed defect (fixed)

Authenticated users without an email address do not get notifications

Reported by: ebouaziz@… Owned by: Jun Omae
Priority: normal Milestone: 1.2.2
Component: notification Version: 1.2.1
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Fix a regression of #12658, ticket notification not sending for authenticated users without email.

API Changes:

Description

This is a regression introduced in #12658

in mail.py in the match_recipient() function in RecipientMatcher :

before #12658, a user with no email address would not appear in self.email_map and use_short_addr or smtp_default_domain options would be applied.

Now, any authenticated user in the database will appear in self.users so we always run the if address in self.users: branch for authenticated users and never the elif ... branch so the use_short_addr or smtp_default_domain options are never applied to these users.

Attachments (0)

Change History (3)

comment:1 by Jun Omae, 3 years ago

Owner: set to Jun Omae
Release Notes: modified (diff)
Status: newassigned

comment:2 by Jun Omae, 3 years ago

Resolution: fixed
Status: assignedclosed

Fixed in [15757] and merged to trunk in [15758]. Thanks for the reporting!

I noticed another issue about the same options, filed #12764.

comment:3 by Ryan J Ollos, 3 years ago

#12766 closed as a duplicate.

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 as closed 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 .
 
Note: See TracTickets for help on using tickets.