Opened 5 years ago

Closed 4 years ago

#1485514 closed Bugs (fixed)

OE sent with UTF-8 bom code

Reported by: joungkyun Owned by:
Priority: 5 Milestone: 0.2.2
Component: PHP backend Version: 0.2-beta
Severity: normal Keywords: utf8 bom
Cc:

Description

On Outlook Express, When using UTF8, OE include UTF-8 BOM code. So, I think that need remove utf8 bom code.

Attachments (4)

include_utf8_bom.eml​ (3.1 KB) - added by joungkyun 5 years ago.
mail source that includes utf8 bom
remove_utf8_bom.patch​ (676 bytes) - added by joungkyun 5 years ago.
include_utf8_bom.jpg​ (44.1 KB) - added by joungkyun 5 years ago.
remove_utf8_bom.jpg​ (40.2 KB) - added by joungkyun 5 years ago.

Download all attachments as: .zip

Change History (11)

Changed 5 years ago by joungkyun

mail source that includes utf8 bom

Changed 5 years ago by joungkyun

Changed 5 years ago by joungkyun

Changed 5 years ago by joungkyun

comment:1 Changed 5 years ago by joungkyun

  • Milestone changed from later to 0.2-stable

I test svn revision 2000.

comment:2 follow-up: Changed 5 years ago by alec

I've loaded attached message and it looks like on remove_utf8_bom.jpg without applying your patch.

comment:3 in reply to: ↑ 2 Changed 5 years ago by joungkyun

Replying to alec:

I've loaded attached message and it looks like on remove_utf8_bom.jpg without applying your patch.

I'm sorry. text/plain part of multi parts on attached message don't have utf8 bom code. But text/html part has utf8 bom code, and when print text/html part, print broken.

Please recheck. Thanks.

comment:4 Changed 5 years ago by alec

Ok, there is BOM, but it's lost after washtml use. So, still works for me.

comment:5 Changed 5 years ago by alec

  • Component changed from MIME parsing to PHP backend
  • Resolution set to fixed
  • Status changed from new to closed

Probably fixed in [5a173887].

comment:6 Changed 4 years ago by tjm74

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Type changed from Feature Requests to Bugs

thomasb's fix missed a case -- need to add \xEF to the group of chars that matches the BOM. There must be a few different BOM indicators? Or someone's dyslexic, since I see \xFE in there...

This is on (for me) line 650 of program/steps/mail/func.inc -- thanks!

comment:7 Changed 4 years ago by alec

  • Milestone changed from 0.2-stable to 0.2.2
  • Resolution set to fixed
  • Status changed from reopened to closed

Please, check [3bde305e] change.

Note: See TracTickets for help on using tickets.