Defect #25269

Emails with no text or html Content not handled properly

Added by Felix Schäfer 8 months ago. Updated 8 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Email receiving
Target version:-
Resolution:Fixed Affected version:

Description

This currently only affects trunk.

The change to the handling of incoming emails committed in r16371 and proposed in #25256 unfortunately improperly handles emails with multipart content but no content in a text or html part of the email. This might be the case if the email is intentionally left blank.

I will provide a patch and a test for that case.

25269-intentionaly_blank_emails.patch Magnifier (1.86 KB) Felix Schäfer, 2017-03-06 17:52

25269-intentionaly_blank_emails.patch Magnifier (3.17 KB) Felix Schäfer, 2017-03-07 08:20


Related issues

Related to Redmine - Defect #25256: Mail parts with empty content should be ignored Closed

Associated revisions

Revision 16379
Added by Jean-Philippe Lang 8 months ago

Emails with no text or html Content not handled properly (#25269, #25256).

Patch by Felix Schäfer.

History

#1 Updated by Felix Schäfer 8 months ago

Attached is a patch with the fix and a test for the case of no content in the html or text parts. Also included are comments provided by my colleague Holger Just to make clear what each part does in what case.

#2 Updated by Go MAEDA 8 months ago

  • Related to Defect #25256: Mail parts with empty content should be ignored added

#3 Updated by Felix Schäfer 8 months ago

My previous patch did not include the fixture email file for the test, sorry for the oversight.

#4 Updated by Jean-Philippe Lang 8 months ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Thanks for this fix.

Also available in: Atom PDF