Edgewall Software

Opened 5 years ago

Last modified 5 years ago

#12759 closed defect

Authenticated users without an email address do not get notifications — at Version 1

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:
Internal 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.

Change History (1)

comment:1 by Jun Omae, 5 years ago

Owner: set to Jun Omae
Release Notes: modified (diff)
Status: newassigned
Note: See TracTickets for help on using tickets.