diff options
author | Tulio Casagrande <tcasagra@thoughtworks.com> | 2016-06-16 17:40:53 -0300 |
---|---|---|
committer | Tulio Casagrande <tcasagra@thoughtworks.com> | 2016-06-16 17:40:53 -0300 |
commit | 18b0c6d8f71972be8bbe7e7ce08176c91a912080 (patch) | |
tree | 4b171740fd6f8cd044de6f9cefb8778d047674cf /service/pixelated/adapter/mailstore/leap_mailstore.py | |
parent | f5cf17f34227f8fe391d769f3cb33aad6de5172e (diff) |
Fix UNNAMED attachment bug
Previously some attachments were being showed as UNNAMED. This change fix it and
creates propers tests for attachments.
See: #608
Diffstat (limited to 'service/pixelated/adapter/mailstore/leap_mailstore.py')
-rw-r--r-- | service/pixelated/adapter/mailstore/leap_mailstore.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/service/pixelated/adapter/mailstore/leap_mailstore.py b/service/pixelated/adapter/mailstore/leap_mailstore.py index 70b53c37..97c95a90 100644 --- a/service/pixelated/adapter/mailstore/leap_mailstore.py +++ b/service/pixelated/adapter/mailstore/leap_mailstore.py @@ -166,7 +166,7 @@ class LeapMail(Mail): def _extract_filename(headers, default_filename='UNNAMED'): - content_disposition = headers.get('Content-Disposition', '') + content_disposition = headers.get('Content-Disposition') or headers.get('content-disposition', '') filename = _extract_filename_from_name_header_part(content_disposition) if not filename: filename = headers.get('Content-Description', '') @@ -363,7 +363,7 @@ class LeapMailStore(MailStore): return self._extract_part_map(part_maps) def _is_attachment(self, part_map, headers): - disposition = headers.get('Content-Disposition', None) + disposition = headers.get('Content-Disposition') or headers.get('content-disposition') content_type = part_map['ctype'] if 'multipart' in content_type: |