0003299: Email notifications have blank lines in header - MantisBT

ID	Project	Category	View Status	Date Submitted	Last Update

0003299	mantisbt	email	public	2003-08-19 08:37	2004-09-12 08:27

Reporter	astax	Assigned To	jlatour
Priority	normal	Severity	major	Reproducibility	always
Status	closed	Resolution	fixed
Product Version	0.18.0a4
Fixed in Version	0.19.0rc1

Summary	0003299: Email notifications have blank lines in header
Description	Here is an quote from email, I've got from Mantis: Return-Path: www-data@.... Delivered-To: astax@..... Received: by ..... (Postfix, from userid 33) id 4BF483D8; Tue, 19 Aug 2003 12:29:21 +0700 (NOVST) To: all_mantis@.... Subject: [Claromentis 0000172]: Opening Excel file frequently causes MSWord to be launched in browser window From: bugreport@.... X-Sender: bugreport@.... X-Mailer: PHP/4.1.2 X-Priority: 3 Content-Type: text/plain; charset=iso-8859-15 Message-Id: 20030819052921.4BF483D8@.... Date: Tue, 19 Aug 2003 12:29:21 +0700 (NOVST) X-UIDL: 7pc!!l_0!!60!!a8f"! The following bug has been RESOLVED. ====================================================================== etc.. all lines are separated with double linebreaks
Tags	No tags attached.

vboctor 2003-08-19 11:53 manager ~0004537	What are your settings for the following configuration options? $g_use_phpMailer $g_phpMailer_method

astax 2003-08-19 12:03 reporter ~0004539	Both are zeros

astax 2003-08-19 12:20 reporter ~0004541	After disabling make_lf_crlf() function (changed it so it replaces "\r\n" to only "\n"), all emails are correct.

brobertson 2003-10-02 04:25 reporter ~0004668	We have also had this problem and commenting out this line $t_headers = make_lf_crlf( $t_headers ); makes it work. We are using qmail, which I gather doesn't like \r in the headers, I don't know about sendmail and others though This is on line 709 in core/email_api.php version : $Id: email_api.php,v 1.61 2003/07/30 02:59:59 beerfrick Exp $

jlatour 2004-08-09 07:38 reporter ~0006863	I'm a little reluctant to put this on the list for 0.19.0 since changing it to just \n might just break it for other MTAs, so it will have to be looked in a bit more carefully than we want to do before release.

astax 2004-08-10 06:04 reporter ~0006904	I've just installed test instance of Mantis, taken latest version from CVS and it seems like this problem if fixed there. I'll wait until 0.19.0 RC released and upgrade our live instance to see if this problem is really fixed.

jlatour 2004-08-10 06:58 reporter ~0006910	OK, thanks.

astax 2004-09-01 23:05 reporter ~0007387	I've upgraded to 0.19.0rc1 and the problem doesn't appear any more. Thank you.