SugarCRM 6.1 mail mangled

[Jashank]

Hi,

Our recently upgraded SugarCRM 6.1 has just thrown up an interesting error. I was first alerted to it when a client reported not being able to access the accept meeting link.

It seems that some mail sent from Sugar inserts extra \r (or ^M, if you prefer) characters at the end of lines, not only causing mail to be mangled, but headers too.

What's doing this, and how can I patch it?

Jashank

[Jashank]

What, no-one else is getting this or has any ideas?

I tried applying the patch mentioned here (which hasn't been fixed upstream), but that didn't resolve it either.

Jashank

[Jashank]

Okay, since there's been no response, here's a (slightly trimmed) message for comparison. The ^M characters denote where a spurious newline appears. The mail servers we're using are sending and routing the mail correctly, but they display messily in Mozilla Thunderbird, Outlook, and Mutt, but sanely on RIM BlackBerry.

Date: Thu, 13 Jan 2011 03:19:06 +0000^M
To: patriciap@XXXXXXXXXXXXXXXXXXXXXXXX.XXX.XX
From: Anita Lush <anital@XXXXXXXXXX.XXX.XX>
Reply-to: Anita Lush <anital@XXXXXXXXXXXXXXXXXXXXXXXX.XXX.XX>
Subject: RE: TEST^M
Message-ID: <71f13be3e468f45fc5f1f1067b624d3c@127.0.0.1>^M
X-Priority: 3^M
X-Mailer: PHPMailer (phpmailer.codeworxtech.com) [version 2.3]^M
MIME-Version: 1.0^M
Content-Type: multipart/alternative;^M
	boundary="b1_71f13be3e468f45fc5f1f1067b624d3c"^M

^M
^M
--b1_71f13be3e468f45fc5f1f1067b624d3c^M
Content-Type: text/plain; charset = "ISO-8859-1"^M
Content-Transfer-Encoding: base64^M
^M
Jm5ic3A7IFRoaXMgaXMgYSByZXBseSBmcm9tIHN1Z2FyLg0KDQoNCg0KDQoNCk9uICAyMDExLTAx^M
LTEzLCAxNDowOSwgUGF0cmljaWEgUGFlY2ggKHBhdHJpY2lhcEBwcm9mZXNzaW9uYWx1dGlsaXR5^M
Ym9hcmQuY29tLmF1KSB3cm90ZToNCg0KIA0KDQotLSANCiANCiANCiANCiANCg0KIA0KIEtpbmQg^M
UmVnYXJkcw0KIA0KIFBhdHJpY2lhIFBhZWNoDQogDQogU2FsZXMgRGlyZWN0b3IgDQogDQoNCiAN^M
CiANCg0KIA0KIHA6IDEzMDAgMjU5IDk0OQ0KIA0KIGQ6IDAyIDg5MTkgMjE0MQ0KIA0KIGY6IDEz^M
MDAgMzMwIDg1OQ0KIA0KIG06IDA0MTkgMjA1IDQ5MA0KIA0KIA0Kd3d3LlByb2Zlc3Npb25hbFV0^M
aWxpdHlCb2FyZC5jb20uYXUNCg0KDQpUaGlzIGVtYWlsIGlzIGludGVuZGVkIG9ubHkgdG8gDQog^M
YmUgDQogcmVhZCBvciB1c2VkIGJ5IHRoZSBhZGRyZXNzZWUuIEl0IGlzIGNvbmZpZGVudGlhbCBh^M
bmQgbWF5IA0KIGNvbnRhaW4gDQogbGVnYWxseSBwcml2aWxlZ2VkIGluZm9ybWF0aW9uLiBJZiB5^M
b3UgYXJlIG5vdCB0aGUgYWRkcmVzc2VlIA0KIGluZGljYXRlZCANCiBpbiB0aGlzIG1lc3NhZ2Ug^M
KG9yIHJlc3BvbnNpYmxlIGZvciBkZWxpdmVyeSBvZiB0aGUgbWVzc2FnZSB0byANCiBzdWNoIA0K^M
IHBlcnNvbiksIG9yIHlvdSBoYXZlIHJlY2VpdmVkIHRoaXMgY29tbXVuaWNhdGlvbiBpbiBlcnJv^M
ciwgeW91IA0KIG11c3Qgbm90IA0KIGNvcHkgb3IgZGlzdHJpYnV0ZSB0aGlzIG1lc3NhZ2Ugb3Ig^M
YW55IHBhcnQgb2YgaXQgb3Igb3RoZXJ3aXNlIA0KIGRpc2Nsb3NlIA0KIGl0cyBjb250ZW50cyB0^M
byBhbnlvbmUuIENvbmZpZGVudGlhbGl0eSBhbmQgbGVnYWwgcHJpdmlsZWdlIGFyZSANCiBub3Qg^M
DQogd2FpdmVkIG9yIGxvc3QgYnkgcmVhc29uIG9mIG1pc3Rha2VuIGRlbGl2ZXJ5IHRvIHlvdS4g^M
Tm8gDQogcmVwcmVzZW50YXRpb24gDQogaXMgbWFkZSB0aGF0IHRoaXMgZW1haWwgb3IgYXNzb2Np^M
YXRlZCBhdHRhY2htZW50cyAoaWYgYW55KSBhcmUgDQogZnJlZSBvZiANCiB2aXJ1c2VzIG9yIG90^M
aGVyIGRlZmVjdHMuIFZpcnVzIHNjYW5uaW5nIGlzIHJlY29tbWVuZGVkIGFuZCBpcyANCiB0aGUg^M
DQogcmVzcG9uc2liaWxpdHkgb2YgdGhlIHJlY2lwaWVudC4NCg0KDQombmJzcDsNCg0KDQoNCiAN^M
CiZuYnNwOw0KDQoNCg0KDQogDQoNCktpbmQgUmVnYXJkcw0KIEFuaXRhIEx1c2gNCiBPZmZpY2Ug^M
TWFuYWdlciANCg0KcDogMTMwMCAyNTkgOTQ5DQogZjombmJzcDsgMTMwMCAzMzAgODU5DQogd3d3^M
LlByb2Zlc3Npb25hbFV0aWxpdHlCb2FyZC5jb20uYXUNClRoaXMgZW1haWwgaXMgaW50ZW5kZWQg^M
b25seSB0byBiZSByZWFkIG9yIHVzZWQgYnkgdGhlIGFkZHJlc3NlZS4gSXQgaXMgY29uZmlkZW50^M
aWFsIGFuZCBtYXkgY29udGFpbiBsZWdhbGx5IHByaXZpbGVnZWQgaW5mb3JtYXRpb24uIElmIHlv^M
dSBhcmUgbm90IHRoZSBhZGRyZXNzZWUgaW5kaWNhdGVkIGluIHRoaXMgbWVzc2FnZSAob3IgcmVz^M
cG9uc2libGUgZm9yIGRlbGl2ZXJ5IG9mIHRoZSBtZXNzYWdlIHRvIHN1Y2ggcGVyc29uKSwgb3Ig^M
eW91IGhhdmUgcmVjZWl2ZWQgdGhpcyBjb21tdW5pY2F0aW9uIGluIGVycm9yLCB5b3UgbXVzdCBu^M
b3QgY29weSBvciBkaXN0cmlidXRlIHRoaXMgbWVzc2FnZSBvciBhbnkgcGFydCBvZiBpdCBvciBv^M
dGhlcndpc2UgZGlzY2xvc2UgaXRzIGNvbnRlbnRzIHRvIGFueW9uZS4gQ29uZmlkZW50aWFsaXR5^M
IGFuZCBsZWdhbCBwcml2aWxlZ2UgYXJlIG5vdCB3YWl2ZWQgb3IgbG9zdCBieSByZWFzb24gb2Yg^M
bWlzdGFrZW4gZGVsaXZlcnkgdG8geW91LiBObyByZXByZXNlbnRhdGlvbiBpcyBtYWRlIHRoYXQg^M
dGhpcyBlbWFpbCBvciBhc3NvY2lhdGVkIGF0dGFjaG1lbnRzIChpZiBhbnkpIGFyZSBmcmVlIG9m^M
IHZpcnVzZXMgb3Igb3RoZXIgZGVmZWN0cy4gVmlydXMgc2Nhbm5pbmcgaXMgcmVjb21tZW5kZWQg^M
YW5kIGlzIHRoZSByZXNwb25zaWJpbGl0eSBvZiB0aGUgcmVjaXBpZW50Lg0K^M
^M
^M
--b1_71f13be3e468f45fc5f1f1067b624d3c^M

[Jashank]

Aha! Found it.

It's a bug in the PHPMailer in SugarCRM 6.1.0CE (version 2.3) -- it uses

 $LF = "\r\n" 
  

which is incorrect; in older versions of PHPMailer, particularly the version we were using in 5.1.0CE (which was 1.7?) it was

 $LF = "\n" 
  

Well, I'm going to file a bug against it.

Jashank