Tickets :: [#8858] Mime part charset for html attachment

6.0.0-beta1

7/12/25

Summary	Mime part charset for html attachment
Queue	IMP
Queue Version	Git master
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	patrick.guiran (at) linagora (dot) com
Created	02/09/2010 (5632 days ago)
Due
Updated	02/10/2010 (5631 days ago)
Assigned
Resolved	02/09/2010 (5632 days ago)
Milestone	5
Patch	Yes

02/10/2010 07:27:41 PM	CVS Commit	Comment #9	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #8858~~: Auto-determine charset for text attachments http://git.horde.org/diff.php/imp/docs/CHANGES?rt=horde-git&r1=8c77e8fa7db5a04ad26f8147155df4e458a8a50d&r2=fdaed514a9f940f7ae5dc42101f9ac81a197263d http://git.horde.org/diff.php/imp/lib/Compose.php?rt=horde-git&r1=9ee46a6a40a4b09ca365659ebf8de750aeb15fcf&r2=fdaed514a9f940f7ae5dc42101f9ac81a197263d

02/09/2010 08:32:10 PM	Michael Slusarz	Comment #8 Assigned to Michael Slusarz State ⇒ Resolved Version ⇒ Git master Milestone ⇒ 5	Reply to this comment
Among other issues, your fix is not backwards compatible and we should be using mime magic to determine the charset, not hardcoded regex strings. For these reasons, this will not be fixed in Horde 3.x. However, this has been fixed for Horde 4.0/IMP 5.0.

02/09/2010 08:30:36 PM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #8858~~: Auto-determine charset for text attachments http://git.horde.org/diff.php/imp/docs/CHANGES?rt=horde-git&r1=8c77e8fa7db5a04ad26f8147155df4e458a8a50d&r2=fdaed514a9f940f7ae5dc42101f9ac81a197263d http://git.horde.org/diff.php/imp/lib/Compose.php?rt=horde-git&r1=9ee46a6a40a4b09ca365659ebf8de750aeb15fcf&r2=fdaed514a9f940f7ae5dc42101f9ac81a197263d

02/09/2010 09:07:54 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #6 New Attachment: thunderbird__with_patch.png	Reply to this comment
Thunderbird : test result with the patch

02/09/2010 09:07:25 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #5 New Attachment: thunderbird__without_patch.png	Reply to this comment
Thunderbird : test result without the patch

02/09/2010 09:05:53 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #4 New Attachment: kmail__with_patch.png	Reply to this comment
Kmail : test result with the patch

02/09/2010 09:05:20 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #3 New Attachment: kmail__without_patch.png	Reply to this comment
Kmail : test result without the patch

02/09/2010 09:03:18 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #2 New Attachment: test.html	Reply to this comment
test file with french words (iso-8859-1)

02/09/2010 09:02:02 AM	patrick (dot) guiran (at) linagora (dot) com	Comment #1 Patch ⇒ Yes State ⇒ New New Attachment: horde-webmail-1.2.4-lng2_patch-html-attachment.patch Milestone ⇒ Queue ⇒ IMP Summary ⇒ Mime part charset for html attachment Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
When a user sends a mail with an HTML file as attachment, the HTML file can't be displayed correctly on mail readers program, due to lack of charset information in the MIME Part. /!\ note /!\ The attached file is a patch for horde-webmail 1.2.4. It has been deployed in production context since 2 months without problem * $Horde: imp/lib/Compose.php,v 1.107.2.86 2009/07/29 06:04:42 slusarz Exp $ * $Horde: framework/MIME/MIME/Magic.php,v 1.52.8.18 2009/01/06 15:23:20 jan Exp $ use case test : 1/ compose a new mail with horde 2/ attach a html file with a charset specified 3/ send the mail 4/ receive the mail on a mail client (thunderbird, kmail, ...) 5/ open the attachment file without save it on disk To check the html file charset, I use the "file" commande on Linux : $ lsb_release -a Distributor ID: Ubuntu Description: Ubuntu 9.10 Release: 9.10 Codename: karmic $ file --mime test.html test.html: text/html; charset=iso-8859-1