If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

mail_delivery_method = 'Test' doesn't generate a valid mbox file

RESOLVED FIXED in Bugzilla 3.0

Status

()

Bugzilla
Email Notifications
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: Frédéric Buclin, Assigned: Frédéric Buclin)

Tracking

3.1.3
Bugzilla 3.0
Bug Flags:
approval +
approval3.0 +

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
We must drop "\n\n---\n\n" and add "From - Mon Aug 20 12:57:39 2007" instead to make it a valid mbox file. This way, we can then use mail_delivery_method = 'Test' for QA and have mailer.testfile being correctly read by email clients such as Thunderbird and SeaMonkey.

This is definitely a must have for QA as that's how I check emails being sent (to fake email addresses). And I don't read encoded strings so I much prefer to let the email client decode them for me. :)

Targetting to 3.0 as that's *really* something everybody testing Bugzilla needs.
(Assignee)

Comment 1

10 years ago
Created attachment 310351 [details] [diff] [review]
patch, v1

Now both Thunderbird and SeaMonkey are happy and can correctly read mailer.testfile.
Attachment #310351 - Flags: review?(justdave)
Comment on attachment 310351 [details] [diff] [review]
patch, v1

obvious.  :)
Attachment #310351 - Flags: review?(justdave) → review+
(Assignee)

Updated

10 years ago
Flags: approval3.0+
Flags: approval+
Keywords: relnote
(Assignee)

Comment 3

10 years ago
tip:

Checking in Bugzilla/Mailer.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Mailer.pm,v  <--  Mailer.pm
new revision: 1.19; previous revision: 1.18
done

3.0.3:

Checking in Bugzilla/Mailer.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Mailer.pm,v  <--  Mailer.pm
new revision: 1.7.2.8; previous revision: 1.7.2.7
done
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Comment 4

9 years ago
Added to the release notes for Bugzilla 3.2 in a patch on bug 432331.
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.