Mark Sapiro tried to write but fumbled his MUA and sent only the
attachment to the list:

>>I just sent you another e-mail with errors from today.  I am going to
>>gzip up all of the log files and send them over to you.  That way you
>>can see everything and hopefully figure out what is going on.
>
>
>I haven't looked at the logs yet, but I know what the problem is. This
>code has been in use since 2.1.11, but it's wrong. The error only
>occurs if bounce_you_are_disabled_warnings = 0 so the member is
>immediately removed.
>
>What happens is we go through all the steps to remove the bouncing
>member including sending the admin and user notice and we do remove
>the member, but then we try to update the bounce info for the removed
>member and encounter the exception. As a result of the exception, the
>updated list is not saved so the member is still there.
>
>The attached patch should fix it.      

-- 
Mark Sapiro <m...@msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

=== modified file 'Mailman/Bouncer.py'
--- Mailman/Bouncer.py  2008-06-20 19:47:28 +0000
+++ Mailman/Bouncer.py  2009-02-15 21:42:10 +0000
@@ -163,8 +163,10 @@
         # We've set/changed bounce info above.  We now need to tell the
         # MemberAdaptor to set/update it.  We do it here in case the
         # MemberAdaptor stores bounce info externally to the list object to
-        # be sure updated information is stored.
-        self.setBounceInfo(member, info)
+        # be sure updated information is stored, but we have to be sure the
+        # member wasn't removed.
+        if self.isMember(member):
+            self.setBounceInfo(member, info)
 
     def disableBouncingMember(self, member, info, msg):
         # Initialize their confirmation cookie.  If we do it when we get the

------------------------------------------------------
Mailman-Users mailing list
Mailman-Users@python.org
http://mail.python.org/mailman/listinfo/mailman-users
Mailman FAQ: http://wiki.list.org/x/AgA3
Searchable Archives: http://www.mail-archive.com/mailman-users%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-users/archive%40jab.org

Security Policy: http://wiki.list.org/x/QIA9

Reply via email to