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: 21249
Status: resolved
Priority: 0/
Queue: Mail-DeliveryStatus-BounceParser

People
Owner: Nobody in particular
Requestors: nonplus [...] gmail.com
Cc:
AdminCc:

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



Subject: Problem with non-standard message/delivery-status
Date: Wed, 30 Aug 2006 22:16:11 -0500
To: bug-Mail-DeliveryStatus-BounceParser [...] rt.cpan.org
From: "Stepan Riha" <nonplus [...] gmail.com>
I've got a bounced email whose message/delivery-status section has an extra blank line. I'm not sure why, although it's possible that the message got munged by "SurfControl E-mail Filter". This could be fixed on line 338 of BounceParser.pm by stripping leading "\n" from $para. Instead of foreach my $para (split /\n\n/, $delivery_status_body) { my $report = Mail::Header->new([split /\n/, $para]); use the following: foreach my $para (split /\n\n/, $delivery_status_body) { $para =~ s/^\n//; my $report = Mail::Header->new([split /\n/, $para]); This shouldn't affect well formed bounces, but would fix the situation where you have an extra blank line between section. Here's the relevant section from the bounce message - note the leading two blank lines following "Content-Type: message/delivery-status;": ----=_NextPart_ST_05_01_24_Wednesday_August_16_2006_16202 Content-Type: message/delivery-status; Action: failed Final-Recipient: rfc822; some_user@ccc.edu Diagnostic-Code: smtp; 554 Service currently unavailable Status: 5.0.0 ----=_NextPart_ST_05_01_24_Wednesday_August_16_2006_16202 Content-Type: message/rfc822;
This should be resolved with the latest release, 1.515 -- rjbs
Subject: Re: [rt.cpan.org #21249] Problem with non-standard message/delivery-status
Date: Tue, 5 Sep 2006 23:38:23 -0500
To: bug-Mail-DeliveryStatus-BounceParser [...] rt.cpan.org
From: "Stepan Riha" <nonplus [...] gmail.com>
I tested the bad bounce with 1.515 and the issue is not resolved. I am attaching a cleaned version (changed email addresses and IPs, redacted message content) of the entire bounced email that demonstrates the problem. - Stepan On 9/5/06, Ricardo Signes via RT <bug-Mail-DeliveryStatus-BounceParser@rt.cpan.org> wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=21249 > > > This should be resolved with the latest release, 1.515 > > -- > rjbs >
-- Stepan Riha nonplus@gmail.com

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

From: wby [...] cpan.org
On Wed Sep 06 00:38:37 2006, nonplus@gmail.com wrote: Show quoted text
> I tested the bad bounce with 1.515 and the issue is not resolved. I > am attaching a cleaned version (changed email addresses and IPs, > redacted message content) of the entire bounced email that > demonstrates the problem.
That's odd - I don't quite understand why after the split on \n{2,} (which should be greedy), there's still a leading \n. But yeah - a version of the fix you suggested seems to work (patch attached). I'll commit this, or figure out a better fix. Should get fixed with the next release (5. I added a test in t/surfcontrol-extra-newline.t and t/corpus/surfcontrol-extra-newline.msg (I used your example, but did a little further sanitizing and removed the Content-Length headers).
Download diff
application/octet-stream 447b

Message body not shown because it is not plain text.

On Thu Sep 28 14:35:20 2006, WBY wrote: Show quoted text
> Should get fixed with the next release (5.
Erm 1.517
Closing.
The comment says closed, but the bug isn't. oops? closing for real -- rjbs