[Mailman-Users] mailman python-2.4 using 96% cpu

[Mark Sapiro]

Goodman, William wrote:
>
>Theses are the high cpu grabbers:
>
>mailman   4764  4751 74 Feb04 ?        17:00:03 /usr/bin/python2.4
>/opt/software/mailman/bin/qrunner --runner=IncomingRunner:0:1 -s
>
>UID        PID  PPID  C STIME TTY          TIME CMD
>mailman   4766  4751 22 Feb04 ?        05:08:33 /usr/bin/python2.4
>/opt/software/mailman/bin/qrunner --runner=OutgoingRunner:0:1 -s
>
>UID        PID  PPID  C STIME TTY          TIME CMD
>mailman   4760  4751  4 Feb04 ?        01:01:34 /usr/bin/python2.4
>/opt/software/mailman/bin/qrunner --runner=BounceRunner:0:1 -s 


OK, thanks.


>Also there's a dead like to the patch location you sent me.


Are you trying to go the URL I sent or are you trying to go to the
first of the two lines your MUA split it into?

><http://mail.python.org/pipermail/mailman-announce/2009-January/000129.h
>tml>,

In any case, here is the patch:

=== modified file 'Mailman/Handlers/Scrubber.py'
--- Mailman/Handlers/Scrubber.py        2008-12-01 04:30:43 +0000
+++ Mailman/Handlers/Scrubber.py        2009-01-12 17:45:14 +0000
@@ -1,4 +1,4 @@
-# Copyright (C) 2001-2008 by the Free Software Foundation, Inc.
+# Copyright (C) 2001-2009 by the Free Software Foundation, Inc.
 #
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License
@@ -167,6 +167,9 @@
     # message by a text (scrubbing).
     del msg['content-type']
     del msg['content-transfer-encoding']
+    if isinstance(charset, unicode):
+        # email 3.0.1 (python 2.4) doesn't like unicode
+        charset = charset.encode('us-ascii')
     msg.set_payload(text, charset)

This patch can also be found at
<http://bazaar.launchpad.net/~mailman-coders/mailman/2.1/revision/1160>

Two other patches you may want to apply are at
<http://bazaar.launchpad.net/~mailman-coders/mailman/2.1/revision/1162>
and
<http://bazaar.launchpad.net/~mailman-coders/mailman/2.1/revision/1166>,
but as I said I will be releasing 2.1.12rc2 in a day or two with these
changes.

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