[Mailman-Developers] Looping digest - mailman bug?
Kaja P. Christiansen
kaja@daimi.au.dk
Mon, 30 Jul 2001 19:09:47 +0200
Hi,
On Wed, 20 Jun 2001 19:12:00 +0200, I wrote:
> Last Monday, one of our mailing lists went mad and sent multiple copies
> of the same digest to the list's digest users.
>
> [...]
>
> The traceback in the errors log shows:
>
> Jun 19 00:01:01 2001 (18618) Traceback (innermost last):
> File "/usr/local/mailman203/Mailman/Handlers/HandlerAPI.py", line 82, in do_pipeline
> func(mlist, msg, msgdata)
> File "/usr/local/mailman203/Mailman/Handlers/Sendmail.py", line 86, in process
> fp.write(msgtext)
> IOError: [Errno 32] Broken pipe
>
> We run Mailman 2.0.3 with Postfix. I addressed this problem earlier
> (March 26th), but there was no reply.
Several of our lists suffered from Mailman's mail bombing and we turned
the digest option off in hope it'll help. It did, for a while, until
the same happened with a non-digest message being send over and over again
(once per minute, by qrunner).
We were able to locate where and how it happens, and to 'reproduce' the
error (in laboratory conditions :-) both under mailman 2.0.3 and 2.0.6.
The mail looping occurs when there is a MIME message with a single . (dot)
in a line; Mailman sends it to sendmail 'as is'. But since sendmail/postfix
interprets a line with single dot as the end of the message, it sends
everything before the dot and exits with 'Broken pipe'. Mailman, however,
still has the message in it's queue and sends it all over again...
When I tried sending non-MIME message with single-dot-line in it to a Mailman
list, there was no looping, but the message body after the single dot was
missing.
Until there is something better, I suggest adding to Sendmail.py a patch
which perhaps is not pretty (it adds a space before the infamous dot),
but it works:
--- ./Mailman/Handlers/Sendmail.py.orig Fri Jul 27 13:40:31 2001
+++ ./Mailman/Handlers/Sendmail.py Fri Jul 27 14:11:46 2001
@@ -31,6 +31,7 @@
import string
import os
+import re
import HandlerAPI
from Mailman import mm_cfg
@@ -80,6 +81,8 @@
msgtext = str(msg)
# cycle through all chunks
failedrecips = []
+ bar = re.compile('\n\.')
+ msgtext = re.sub(bar, '\n .', msgtext, 0)
for chunk in recipchunks:
# TBD: SECURITY ALERT. This invokes the shell!
fp = os.popen(cmd + chunk, 'w')
Kaja