Defect #15684

MailHandler : text/plain attachments are added to description

Added by Pavel Brych almost 4 years ago. Updated almost 4 years ago.

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

0%

Category:Email receiving
Target version:2.4.2
Resolution:Fixed Affected version:2.4.0

Description

When handling email messages with text/plain file attached, it's contents is attached to issue description text (file from email is also properly attached to newly created issue, as it should be).

I believe that it may be caused by patch #13646, which combines all text/plain message parts to issue description and not distinguish between text part and file attachment?.

Example of problematic mail (sent from Thunderbird client):

--===============0191659920==
Content-Type: text/plain; charset="utf-8" 
MIME-Version: 1.0
Content-Transfer-Encoding: base64

SGzDocWhZW7DrSBvIGNoeWLEmyB2IHByb2dyYW11IEhPUkVTCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09CgpEYXR1bTogMTEuMTIuMjAxMy... proper message body text continues

--===============0191659920==
Content-Type: text/plain; charset="utf-8" 
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="log_message.txt" 

RGF0ZTogICAgMTEuMTIuMjAxMyAxODoxODowMQpNZXNzYWdlOiBBdHRyaWJ1dGVFcnJvcjogJ2Z1
bmN0aW9uJyBvYmplY3QgaGFzIG5vIGF0dHJpYnV0ZSAnaXNEZWFkJwpUeXBlOiAgICA0C... attachment data continues

Problems started when upgraded to Redmine 2.4.0


Related issues

Related to Redmine - Patch #13646: Fix handling multiple text parts in email Closed

Associated revisions

Revision 12420
Added by Jean-Philippe Lang almost 4 years ago

Fixed that text/plain attachments are added to issue description (#15684).

History

#1 Updated by Toshi MARUYAMA almost 4 years ago

  • Related to Patch #13646: Fix handling multiple text parts in email added

#2 Updated by Toshi MARUYAMA almost 4 years ago

  • Target version set to 2.4.2

#3 Updated by Jean-Philippe Lang almost 4 years ago

Could you please attach a full example email so we can test this?

#4 Updated by Jean-Philippe Lang almost 4 years ago

  • Subject changed from Wrong text/plain attachment handling when creating issue from mail to MailHandler : text/plain attachments are added to description
  • Status changed from New to Resolved
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Fixed in r12420, thanks for pointing this out.

#5 Updated by Jean-Philippe Lang almost 4 years ago

  • Status changed from Resolved to Closed

Merged.

Also available in: Atom PDF