Modify

Opened 13 years ago

Closed 7 years ago

#8796 closed defect (fixed)

notifications also get sent to the smtp_always_cc address

Reported by: Brecht Machiels Owned by: Steffen Hoffmann
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: email notification private
Cc: Ryan J Ollos Trac Release: 0.12

Description

I have the notification/smtp_always_cc option in trac.ini set to a mailing list address so that people can easily follow ticket changes. When AccountManager is sending notifications of user registration/deletion, these are also send to the smtp_always_cc address, in addition to the address(es) specified for AccountManager notifications. However, I don't want AccountManager to send to the smtp_always_cc address.

I have tried to adjust acct_mgr/notification.py to temporarily empty the smtp_always_cc option in the configuration, but this didn't seem to work. Probably I'm misunderstanding what is going on?

Attachments (1)

trac-accountmanager-fix-always_cc.patch (756 bytes) - added by arthur@… 12 years ago.
Patch to fix this issue. The implementation is based on the SingleUserNotification below.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 13 years ago by Steffen Hoffmann

Keywords: email notification private added

This isn't the only notification-related issue. Please give me the chance to look at it after I've finished the currently prepared release.

Changed 12 years ago by arthur@…

Patch to fix this issue. The implementation is based on the SingleUserNotification below.

comment:2 Changed 12 years ago by Ryan J Ollos

Cc: Ryan J Ollos added; anonymous removed

comment:3 Changed 11 years ago by mike bayer

Trac Release: 0.121.0

this workaround worked throughout trac 0.12 but now fails in trac 1.0.1. This issue makes it impossible for a trac to be sending ticket emails to a public list since all the username notifications go there too.

comment:4 Changed 11 years ago by mike bayer

nevermind, it still works in 1.0, but you guys really need to commit this.

comment:5 in reply to:  4 Changed 11 years ago by Steffen Hoffmann

Trac Release: 1.00.12

Replying to mike_mp@zzzcomputing.com:

nevermind, it still works in 1.0, but you guys really need to commit this.

Point taken. But how about joining us? I could really use help with this high-profile plugin. Furthermore you could already assist by not changing ticket properties meant to help other developers spending their voluntary time more efficiently. Thanks for consideration and taking care.

comment:6 Changed 11 years ago by anonymous

whoops, didn't mean to change the release. The patch that's already attached works as is, you can commit it. Not sure what your testing procedure is.

comment:7 Changed 9 years ago by Steffen Hoffmann

In 14276:

AccountManagerPlugin: Don't notify users other than these in 'account_changes_notify_addresses' option, refs #8796.

Thanks to Arthur for proposing the change to fix this issue.

comment:8 Changed 9 years ago by Steffen Hoffmann

Status: newaccepted

comment:9 Changed 7 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.