#12759 closed defect (fixed)
Authenticated users without an email address do not get notifications
| Reported by: | 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.
Attachments (0)
Change History (3)
comment:1 by , 9 years ago
| Owner: | set to |
|---|---|
| Release Notes: | modified (diff) |
| Status: | new → assigned |
comment:2 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |



Fixed in [15757] and merged to trunk in [15758]. Thanks for the reporting!
I noticed another issue about the same options, filed #12764.