diff options
| author | Mark Sapiro <mark@msapiro.net> | 2021-03-06 11:11:23 -0800 |
|---|---|---|
| committer | Mark Sapiro <mark@msapiro.net> | 2021-03-06 11:11:23 -0800 |
| commit | 416d2e6ef003869486dcf6d212133511114767be (patch) | |
| tree | 03b7ba60b332a79b7bf75c105b0b68d38c7bb7f1 | |
| parent | d2ca3472e083e435e812cbc3247cccf372cba906 (diff) | |
| download | mailman2-416d2e6ef003869486dcf6d212133511114767be.tar.gz mailman2-416d2e6ef003869486dcf6d212133511114767be.tar.xz mailman2-416d2e6ef003869486dcf6d212133511114767be.zip | |
Fix bug in prior commit.
| -rw-r--r-- | Mailman/Handlers/Scrubber.py | 4 |
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) |