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 <[email protected]> The highway is for gamblers,
San Francisco Bay Area, California better use your sense - B. Dylan
------------------------------------------------------
Mailman-Users mailing list [email protected]
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
