"CVR" == Chuq Von Rospach <chuqui@plaidworks.com> writes:
CVR> There's a bug in the bounce processing in B4. I just upgraded CVR> my productio server (finally!) to mailman, and I'm getting CVR> these in the error log: This can only happen if the bounce message has a bogus Original-Recipient: or Final-Recipient: header. If it's missing the address type field, the split will fail. In that case, I'm inclined not to guess at what the MTA was trying to say, and to return None, meaning we couldn't extract an address from the header. Attached is a patch, but it would still be great if you had a sample of the message that broke this so I could add it to my test suite. -Barry -------------------- snip snip -------------------- Index: DSN.py =================================================================== RCS file: /cvsroot/mailman/mailman/Mailman/Bouncers/DSN.py,v retrieving revision 1.5 diff -u -r1.5 DSN.py --- DSN.py 2000/06/08 03:44:38 1.5 +++ DSN.py 2000/07/17 21:59:54 @@ -25,7 +25,11 @@ def parseaddr(val): - atype, addr = string.split(val, ';') + try: + atype, addr = string.split(val, ';') + except ValueError: + # Bogus format for Original-Recipient: or Final-Recipient: fields + return None if string.lower(string.strip(atype)) <> 'rfc822': return None addr = string.strip(addr)