Incoming emails with attachments are not processed

(Paul Apostolos) #1

As I documented in this post

Incoming email (new topics or replies) with an attachment of any size or type is simply not processed by Discourse.

No notification is sent to the sender or the site admin that the email is not processed, it just fails.

(Jeff Atwood) #2

Yep this is on our list. @neil maybe you can get to it this week? Attachments should just be stripped and ignored for now, the current silent failure mode is pretty bad,

(Neil Lalonde) #3

Emails with attachments are falling into this case here: discourse/poll_mailbox.rb at master · discourse/discourse · GitHub

      rescue Email::Receiver::ProcessingError
        # all other ProcessingErrors are ok to be dropped

The ProcessingError comes from a Email::Receiver::EmptyEmailError. For some reason, these emails are seen as being completely empty. We must be parsing these emails wrong somehow. Investigating.

(Neil Lalonde) #4

I pushed a fix for this today. The attachment is still being thrown away, but the email will cause a post or topic to be created. Give it a try.

Stripping incoming email attachments doesn't notify sender
(Jeff Atwood) #5