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

6.0.0-git

2020-10-27

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	2010-02-09 (3913 days ago)
Due
Updated	2010-02-10 (3912 days ago)
Assigned
Resolved	2010-02-09 (3913 days ago)
Milestone	5
Patch	Yes

2010-02-10 19:27:41	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

2010-02-09 20:32:10	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.

2010-02-09 20:30:36	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

2010-02-09 09:07:54	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

2010-02-09 09:07:25	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

2010-02-09 09:05:53	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

2010-02-09 09:05:20	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

2010-02-09 09:03:18	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)

2010-02-09 09:02:02	patrick (dot) guiran (at) linagora (dot) com	Comment #1 Type ⇒ Enhancement State ⇒ New Priority ⇒ 1. Low Summary ⇒ Mime part charset for html attachment Queue ⇒ IMP Milestone ⇒ Patch ⇒ Yes New Attachment: horde-webmail-1.2.4-lng2_patch-html-attachment.patch	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