[Bug 1096066] [NEW] mailman raises exception if passed message not contains text/plain message part

[Robert Niederreiter]

Public bug reported:

If message sent to mailman not contains a text/plain message part (i.e.
only text/html), an error is raised and the message gets swallowed.

I fixed this this way in my setup:

=== modified file 'src/mailman/rules/approved.py'
--- src/mailman/rules/approved.py	2012-07-02 20:08:58 +0000
+++ src/mailman/rules/approved.py	2013-01-04 13:31:09 +0000
@@ -73,6 +73,8 @@
             stripped = False
             for part in typed_subpart_iterator(msg, 'text', 'plain'):
                 break
+            if not part:
+                return False
             payload = part.get_payload(decode=True)
             if payload is not None:
                 charset = part.get_content_charset('us-ascii')

regards, robert

** Affects: mailman
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Mailman
Coders, which is subscribed to GNU Mailman.
https://bugs.launchpad.net/bugs/1096066

Title:
  mailman raises exception if passed message not contains text/plain
  message part

To manage notifications about this bug go to:
https://bugs.launchpad.net/mailman/+bug/1096066/+subscriptions