aboutsummaryrefslogtreecommitdiffstats
path: root/Mailman
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2021-03-06 11:11:23 -0800
committerMark Sapiro <mark@msapiro.net>2021-03-06 11:11:23 -0800
commit416d2e6ef003869486dcf6d212133511114767be (patch)
tree03b7ba60b332a79b7bf75c105b0b68d38c7bb7f1 /Mailman
parentd2ca3472e083e435e812cbc3247cccf372cba906 (diff)
downloadmailman2-416d2e6ef003869486dcf6d212133511114767be.tar.gz
mailman2-416d2e6ef003869486dcf6d212133511114767be.tar.xz
mailman2-416d2e6ef003869486dcf6d212133511114767be.zip
Fix bug in prior commit.
Diffstat (limited to 'Mailman')
-rw-r--r--Mailman/Handlers/Scrubber.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/Mailman/Handlers/Scrubber.py b/Mailman/Handlers/Scrubber.py
index 16ce3b02..a498f814 100644
--- a/Mailman/Handlers/Scrubber.py
+++ b/Mailman/Handlers/Scrubber.py
@@ -202,8 +202,8 @@ def process(mlist, msg, msgdata=None):
# 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)