Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 84113
Status: open
Priority: 0/
Queue: Email-Address
People
Owner: Nobody in particular
Requestors: RINGERC [...] cpan.org
Cc: ruz [...] bestpractical.com
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Mar 21 03:16:24 2013 RINGERC [...] cpan.org - Ticket created
CC: "Ruslan Zakirov" <ruz [...] bestpractical.com>
Subject: No way to recover skipped non-RFC822 parts
When parsing email address lists it can be desirable to recover those parts that could not be recognised and extracted as email addresses. Good reasons to do this are: * Reporting good error messages to the user; and * Handling unmatched parts via secondary lookups In my particular case I want to treat unmatched parts (which may not contain quotes or commas) as local user names and look them up in a database to get their email addresses for an Request Tracker (RT) module. It appears that Email::Address::List is a version of Email::Address adapted to do just this. Would you consider adopting that module's parse method?
  Thu Mar 21 07:37:53 2013 ruz [...] bestpractical.com - Correspondence added
Subject: Re: [rt.cpan.org #84113] No way to recover skipped non-RFC822 parts
Date: Thu, 21 Mar 2013 15:37:32 +0400
To: bug-Email-Address [...] rt.cpan.org
From: Ruslan Zakirov <ruz [...] bestpractical.com>
On Thu, Mar 21, 2013 at 11:16 AM, Craig Ringer via RT <bug-Email-Address@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=84113 > > > When parsing email address lists it can be desirable to recover those parts that could not be recognised and extracted as email addresses. > > Good reasons to do this are: > > * Reporting good error messages to the user; and > * Handling unmatched parts via secondary lookups > > In my particular case I want to treat unmatched parts (which may not contain quotes or commas) as local user names and look them up in a database to get their email addresses for an Request Tracker (RT) module. > > It appears that Email::Address::List is a version of Email::Address adapted to do just this. Would you consider adopting that module's parse method?
I wrote Email::Address::List exactly for the purpose of parsing address strings in emails and switch RT to use ::List module. A branch exist in RT's repo that uses the module. It should be reviewed and merged into master to be in RT 4.2. I don't mind merging two modules together into Email::Address and adding parse_line class method that does line parsing. If maintainers of Email::Address don't mind then somebody should do the merge. -- Best regards, Ruslan.
  Thu Mar 21 07:37:55 2013 The RT System itself - Status changed from 'new' to 'open'
  Fri Aug 02 10:48:21 2013 rjbs [...] cpan.org - Correspondence added
On 2013-03-21 07:37:53, ruz@bestpractical.com wrote: Show quoted text
> I don't mind merging two modules together into Email::Address and > adding parse_line class method that does line parsing. If maintainers > of Email::Address don't mind then somebody should do the merge.
Sounds good to me. If I don't get a patch before then, I will do this myself on my *next* round of reviewing bugs on this dist. -- rjbs
  Mon Aug 26 15:51:14 2013 ruz [...] bestpractical.com - Correspondence added
CC: RINGERC [...] cpan.org
Subject: Re: [rt.cpan.org #84113] No way to recover skipped non-RFC822 parts
Date: Mon, 26 Aug 2013 23:51:01 +0400
To: bug-Email-Address [...] rt.cpan.org
From: Ruslan Zakirov <ruz [...] bestpractical.com>
On Fri, Aug 2, 2013 at 6:48 PM, Ricardo Signes via RT < bug-Email-Address@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=84113 > > > On 2013-03-21 07:37:53, ruz@bestpractical.com wrote:
> > I don't mind merging two modules together into Email::Address and > > adding parse_line class method that does line parsing. If maintainers > > of Email::Address don't mind then somebody should do the merge.
> > Sounds good to me. If I don't get a patch before then, I will do this > myself on my *next* round of reviewing bugs on this dist. >
It's a bit harder than just moving a function. I started from re-using regexps, but ended up with my own set that supports some additional obsolete syntax and gives me sub-strings capturing where I need it. -- Show quoted text
> rjbs >
-- Best regards, Ruslan.