[ mailman-Bugs-1099138 ] Scrubber.py traceback

[SourceForge.net]

Bugs item #1099138, was opened at 2005-01-09 22:36
Message generated for change (Comment added) made by otaylor
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1099138&group_id=103

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Owen Taylor (otaylor)
Assigned to: Nobody/Anonymous (nobody)
Summary: Scrubber.py traceback

Initial Comment:
Outgoing mail on one of our lists on mail.gnome.org was
being held up because of a error that was occuring
sending out digests.

Jan 09 16:07:24 2005 (22410) Uncaught runner exception:
len() of unsized object
Jan 09 16:07:24 2005 (22410) Traceback (most recent
call last):
  File "/var/mailman/Mailman/Queue/Runner.py", line
111, in _oneloop
    self._onefile(msg, msgdata)
  File "/var/mailman/Mailman/Queue/Runner.py", line
167, in _onefile
    keepqueued = self._dispose(mlist, msg, msgdata)
  File "/var/mailman/Mailman/Queue/IncomingRunner.py",
line 130, in _dispose
    more = self._dopipeline(mlist, msg, msgdata, pipeline)
  File "/var/mailman/Mailman/Queue/IncomingRunner.py",
line 153, in _dopipeline
    sys.modules[modname].process(mlist, msg, msgdata)
  File "/var/mailman/Mailman/Handlers/ToDigest.py",
line 91, in process
    send_digests(mlist, mboxfp)
  File "/var/mailman/Mailman/Handlers/ToDigest.py",
line 132, in send_digests
    send_i18n_digests(mlist, mboxfp)
  File "/var/mailman/Mailman/Handlers/ToDigest.py",
line 306, in send_i18n_digests
    msg = scrubber(mlist, msg)
  File "/var/mailman/Mailman/Handlers/Scrubber.py",
line 262, in process
    size = len(payload)
TypeError: len() of unsized object

Unfortunately, I don't know what mail was triggering
this...
was trying to get the mailing list going and didn't want to
spend a lot of time investigating.

It's also not immediately clear to me how payload could
end up as None there... from reading the code in 
email.Message maybe part._payload was None due
to an earlier problem.

I'll attach a workaround patch...  the if: also covers
the call to save_attachment() since that also
tracebacked when I just 
fixed the call to len().

Anyways, filing in case it makes sense to someone more
familiar with the guts of email.Message.

----------------------------------------------------------------------

>Comment By: Owen Taylor (otaylor)
Date: 2005-01-09 23:15

Message:
Logged In: YES 
user_id=63124

Oh, forgot to add say, this is python-2.2.3 and mailman-2.1.5.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1099138&group_id=103