------------------------------------------------------------
revno: 1866
committer: Mark Sapiro <m...@msapiro.net>
branch nick: 2.1
timestamp: Sat 2021-03-06 11:11:23 -0800
message:
  Fix bug in prior commit.
modified:
  Mailman/Handlers/Scrubber.py


--
lp:mailman/2.1
https://code.launchpad.net/~mailman-coders/mailman/2.1

Your team Mailman Checkins is subscribed to branch lp:mailman/2.1.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.1/+edit-subscription
=== modified file 'Mailman/Handlers/Scrubber.py'
--- Mailman/Handlers/Scrubber.py	2021-03-06 02:24:12 +0000
+++ Mailman/Handlers/Scrubber.py	2021-03-06 19:11:23 +0000
@@ -202,8 +202,8 @@
             # MAS: Content-Disposition is not a good test for 'attached'.
             # RFC 2183 sec. 2.10 allows Content-Disposition on the main body.
             # Make it specifically 'attachment'.
-            if part.get('content-disposition').lower() == 'attachment' and \
-               not part.get_content_charset():
+            if (part.get('content-disposition', '').lower() == 'attachment'
+                    and not part.get_content_charset()):
                 omask = os.umask(002)
                 try:
                     url = save_attachment(mlist, part, dir)

_______________________________________________
Mailman-checkins mailing list -- mailman-checkins@python.org
To unsubscribe send an email to mailman-checkins-le...@python.org
https://mail.python.org/mailman3/lists/mailman-checkins.python.org/
Member address: arch...@jab.org

Reply via email to