coddingtonbear / django-mailbox

Import mail from POP3, IMAP, local email mailboxes or directly from Postfix or Exim4 into your Django application automatically.
MIT License
357 stars 166 forks source link

Fix message/rfc822 attachment processing #230

Closed milj closed 10 months ago

milj commented 4 years ago

Hello, We have recently stumbled upon this issue: https://github.com/coddingtonbear/django-mailbox/issues/102. I noticed that https://github.com/coddingtonbear/django-mailbox/pull/103 is still not merged so I decided to do a re-make. I hope it's good enough to be merged.

pfouque commented 10 months ago

@Pietro395 I found this neat PR that was already approved by @coddingtonbear but not merged... The PR looks pretty safe, complete (with test and sample), easy to test and may solve multiple issues at once. @Pietro395 do you think you can give it a QA on your environment so we can merge it, 🙏?

Pietro395 commented 10 months ago

@Pietro395 I found this neat PR that was already approved by @coddingtonbear but not merged... The PR looks pretty safe, complete (with test and sample), easy to test and may solve multiple issues at once. @Pietro395 do you think you can give it a QA on your environment so we can merge it, 🙏?

I tested it in my test environment and everything looks as expected immagine

Thank you for your contribution @milj