Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Mail-DeliveryStatus-BounceParser CPAN distribution.

Report information
The Basics
Id: 99506
Status: patched
Priority: 0/
Queue: Mail-DeliveryStatus-BounceParser
People
Owner: Nobody in particular
Requestors: mintywalker [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.534
Fixed in: (no value)

History Show all quoted text
  Wed Oct 15 10:00:36 2014 mintywalker [...] gmail.com - Ticket created
Subject: Missing <> on email address in bounce
We've started seeing errors like this: Use of uninitialized value $email in hash element at /usr/share/perl5/Mail/DeliveryStatus/BounceParser.pm line 1331. Use of uninitialized value $email in hash element at /usr/share/perl5/Mail/DeliveryStatus/BounceParser.pm line 1332. I believe this is because the code is failing to parse the email address, and thus throwing an undef warning. I added the code to github & created a branch with a failing test & a patch that fixes this error. # 1.534 https://github.com/minty/Mail-DeliveryStatus-BounceParser # Test & Patch https://github.com/minty/Mail-DeliveryStatus-BounceParser/tree/addy You can see the details here: https://github.com/minty/Mail-DeliveryStatus-BounceParser/compare/addy If you'd prefer the patch in another format, please let me know. ps. The bug will persist if for any message it cannot set $email correctly. I think setting $email = '' would silence the warning, but I'm not clear if this is a wise patch or not.
  Tue Oct 21 17:36:04 2014 MSTEVENS [...] cpan.org - Correspondence added
Released your patch in 1.535. I keep the repo on gitorious: https://gitorious.org/mail-deliverystatus-bounceparser-fork/mail-deliverystatus-bounceparser-fork
  Tue Oct 21 17:36:05 2014 The RT System itself - Status changed from 'new' to 'open'
  Tue Oct 21 17:36:05 2014 MSTEVENS [...] cpan.org - Status changed from 'open' to 'patched'
  Wed Oct 22 05:32:21 2014 MSTEVENS [...] cpan.org - Correspondence added
On Tue Oct 21 17:36:04 2014, MSTEVENS wrote: Show quoted text
> Released your patch in 1.535. > > I keep the repo on gitorious: > > https://gitorious.org/mail-deliverystatus-bounceparser-fork/mail- > deliverystatus-bounceparser-fork
Missing file in 1.535, rereleased as 1.536.