On 07/03/2013 07:40 AM, Chris Nulk wrote: > > On 7/2/2013 5:23 PM, Mark Sapiro wrote: >> On 07/02/2013 03:09 PM, Chris Nulk wrote:
>> >> Not necessary to lowercase sender here as msg.get_senders() always >> returns lowercased addresses unless called with a preserve_case argument >> with a True value. > > Thanks. It was more of a just to make sure thing. OK. > Actually, I code I was referring to was in Read_GlobalBan_File: Yes, I understood that. I didn't comment on the change to Read_GlobalBan_File as I had no comment. > Okay. Makes sense. Here is the modified do_discard_globalban code: > > def do_discard_globalban(mlist, msg, sender): > # forward discarded message to site administrator(s) if defined > # in mm_cfg.GLOBALBANLIST_NOTIFY > notifylist = [] > if mm_cfg.GLOBALBANLIST_NOTIFY: > notifylist.append(mm_cfg.GLOBALBANLIST_NOTIFY) You might consider making mm_cfg.GLOBALBANLIST_NOTIFY a list of 1 or more addresses in which case the above would be if mm_cfg.GLOBALBANLIST_NOTIFY: notifylist.extend(mm_cfg.GLOBALBANLIST_NOTIFY) or you could get fancy and test if it's a list or a string and extend or append accordingly. -- Mark Sapiro <m...@msapiro.net> The highway is for gamblers, San Francisco Bay Area, California better use your sense - B. Dylan ------------------------------------------------------ Mailman-Users mailing list Mailman-Users@python.org http://mail.python.org/mailman/listinfo/mailman-users Mailman FAQ: http://wiki.list.org/x/AgA3 Security Policy: http://wiki.list.org/x/QIA9 Searchable Archives: http://www.mail-archive.com/mailman-users%40python.org/ Unsubscribe: http://mail.python.org/mailman/options/mailman-users/archive%40jab.org