Ticket #2766 (closed defect: fixed)
Opened 6 years ago
Last modified 4 years ago
Trac treats usernames with '@' as email when sending notifications
| Reported by: | wfragg@… | Owned by: | eblot |
|---|---|---|---|
| Priority: | high | Milestone: | 0.11 |
| Component: | notification | Version: | 0.9.4 |
| Severity: | major | Keywords: | |
| Cc: | |||
| Release Notes: | |||
| API Changes: | |||
Description
If the username contains the '@' character (e.g, user@…, this is case when using users from the Windows Active Directory), Trac treats it as a e-mail address and do not uses the adress set in the user settings.
Attachments
Change History
Changed 6 years ago by wfragg@…
- Attachment notification.patch added
comment:1 Changed 6 years ago by wfragg@…
Any progress?
comment:3 Changed 6 years ago by eblot
#3392 has been marked as a duplicate.
comment:4 Changed 5 years ago by cboos
- Component changed from general to notification
- Milestone set to 0.12
- Owner changed from jonas to eblot
comment:5 Changed 5 years ago by eblot
- Priority changed from normal to high
- Severity changed from normal to major
- Status changed from new to assigned
This issue has been raised several times and should be easy to fix.
comment:6 Changed 5 years ago by eblot
- Keywords notification email removed
- Milestone changed from 0.12 to 0.11
- Resolution set to fixed
- Status changed from assigned to closed
Should be fixed up in [5508]
Add the following option:
[notification] ignore_domains = SOMEDOMAIN.NSK
comment:7 Changed 4 years ago by clinton.roy@…
Ignore domains don't quite work for me in my kerberos setup, what I'd like to have happen is that the kerberos domain get stripped from the login, then the default_domain get appended.
Currently I've got a one line patch in notification.py:get_smtp_address() that achieves what I'm after:
address = address.replace('@…', '@…')
Possible fix