Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Email-Simple CPAN distribution.

Report information
The Basics
Id: 28050
Status: rejected
Priority: 0/
Queue: Email-Simple

People
Owner: Nobody in particular
Requestors: ccl [...] ccl.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC: jkl [...] ccl.net, janl [...] speakeasy.net
Subject: Bug (Feature?) of Email::Simple::Header
Date: Sun, 8 Jul 2007 17:59:56 -0400 (EDT)
To: bug-Email-Simple [...] rt.cpan.org
From: Computational Chemistry List <ccl [...] ccl.net>
I wanted to use your Email::Simple, Email::MIME to parse/modify messages. However, the module, as it is, will mess up the first header line of UNIX email message, namely the Envelope From line: Show quoted text
> From some@email DATE as DW Mon HH:MM:SS YYYY,
e.g.: Show quoted text
> From chemistry-request@ccl.net Sat Jun 30 19:12:47 2007
when modified message is reconstituted by the as_string method. To fix it and include support for this line explicitly in the Email::Simple::Header module (VERSION = '1.999';) I made small changes and created the patch files. diff Header.pm.OLD Header.pm.NEW > patch_back.txt diff Header.pm.NEW Header.pm.OLD > patch.txt I include forward and reverse patch files as attachments. Jan Labanowski jkl@ccl.net, janl@speakeasy.net

Message body is not shown because sender requested not to inline it.

Message body is not shown because sender requested not to inline it.

The "From " line is not part of the email message. It is part of the mbox storage format. You should be using a reader/writer for mbox. There are many, like Email::Folder::Mbox and Mail::Box::Mbox. -- rjbs
Subject: Re: [rt.cpan.org #28050] Bug (Feature?) of Email::Simple::Header
Date: Fri, 13 Jul 2007 09:43:21 -0400 (EDT)
To: bug-Email-Simple [...] rt.cpan.org
From: Computational Chemistry List <ccl [...] ccl.net>
I do email processing for last 20 years. This is one of these issues: "Make it useful or make it right". You chose to make it right. And I am still not certain if you really made it right, since messages with the From_ line are being processed without error by your module and create messed up header line like Show quoted text
> From my@mail.com Thu Jun 12, 11: 23:33 2006
(i.e., you module sticks the space after first colon and creates bogus header entry) after reconstitution. If you want to be right, you should report the error, like: "No spaces allowed in header line field name" or something. You just wasting time of many people, since, IN PRACTICE, all UNIX messages have the top From_ line. At least, please correct the POD and tell people that when they are on UNIX, they should not use your module or need to do some elaborate preprocessing that makes using your module questionable. I will not use your module, and I will write my own stuff based on MIME::Light. Jan -- Jan Labanowski CCL.NET Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=28050 > > > The "From " line is not part of the email message. It is part of the > mbox storage format. You should be using a reader/writer for mbox. > There are many, like Email::Folder::Mbox and Mail::Box::Mbox. > > -- > rjbs >
Subject: Re: [rt.cpan.org #28050] Bug (Feature?) of Email::Simple::Header
Date: Fri, 13 Jul 2007 10:57:42 -0400
To: Computational Chemistry List via RT <bug-Email-Simple [...] rt.cpan.org>
From: Ricardo SIGNES <rjbs [...] cpan.org>
* Computational Chemistry List via RT <bug-Email-Simple@rt.cpan.org> [2007-07-13T09:43:45] Show quoted text
> I will not use your module, and I will write my own stuff based on > MIME::Light.
MIME::Lite (which I assume you mean) is probably the buggiest email module on the CPAN. I suggest you start with Mail::Box or MIME::Entity. -- rjbs
Unix also teaches us to use the right tool for the job. The right tool for reading an email message may be Email::Simple. The right tool for reading an mbox is NOT Email::Simple. I do agree that a warning or exception should be thrown when encountering a bogus header. Unfortunately, Email::Simple is well-established in use, and its acceptance of this kind of input predates my control of it. Adding new constraints to what is accepted must be done slowly and with warnings. This is in the works. -- rjbs