Bug #27154 for Net-Server-Mail: Broken PIPE: QUIT command in body data

Wed May 16 10:46:28 2007 sebastiano.piccoli [...] staff.dada.net - Ticket created

Subject:	Broken PIPE: QUIT command in body data
Date:	Wed, 16 May 2007 16:44:53 +0200
To:	bug-Net-Server-Mail [...] rt.cpan.org
From:	Sebastiano Piccoli <sebastiano.piccoli [...] dada.net>

Hi, Net::Server::Mail version 0.15 doesn't treat correctly QUIT termination command when pipeling is active. When QUIT is sent without waiting server notification, we have found QUIT string concatenated with body data. We solved that issue by replacing: ($data = "$self->{last_chunk}$data") =~ s/(\r?\n)\.\r?\n/$1/s; with: $data = "$self->{last_chunk}$data"; ($data = $`) =~ s/\r?\n\.\r?\n//mg; in sub data_format(). Regards Raffaello Torraco, Sebastiano Piccoli & Valerio Paolini

Thu May 24 04:11:33 2007 raffaello.torraco [...] staff.dada.net - Correspondence added

Subject:	Re: [rt.cpan.org #27154] Broken PIPE: QUIT command in body data
Date:	Thu, 24 May 2007 10:08:12 +0200
To:	bug-Net-Server-Mail [...] rt.cpan.org
From:	Raffaello Torraco <raffaello.torraco [...] staff.dada.net>

Hi, The better solution to solve this bug is replacing: ($data = "$self->{last_chunk}$data") =~ s/(\r?\n)\.\r?\n/$1/s; with ($data = "$self->{last_chunk}$data") =~s/(\r?\n)\.\r?\n(QUIT\r?\n)?/$1/s; in sub data_part of Net/Server/Mail/SMTP.pm the previus solution had problem with Quoted-Printable (in bodydata remain the character of 'softlinebreak'). Regards, Raffaello Torraco, Sebastiano Piccoli, Valerio Paolini

Thu May 24 04:11:35 2007 The RT System itself - Status changed from 'new' to 'open'

Sat May 26 03:55:02 2007 x.guimard [...] free.fr - Correspondence added 5 min

RT-Send-CC:

raffaello.torraco [...] staff.dada.net,sebastiano.piccoli [...] staff.dada.net

Le Jeu. Mai. 24 04:11:33 2007, raffaello.torraco@staff.dada.net a écrit : Show quoted text

> Hi, > > The better solution to solve this bug is replacing: > > ($data = "$self->{last_chunk}$data") =~ s/(\r?\n)\.\r?\n/$1/s; > > with > > ($data = "$self->{last_chunk}$data") =~s/(\r?\n)\.\r?\n(QUIT\r?\n)?/$1/s; > > in sub data_part of Net/Server/Mail/SMTP.pm > > the previus solution had problem with Quoted-Printable (in bodydata > remain the character of 'softlinebreak'). > > Regards, > Raffaello Torraco, Sebastiano Piccoli, Valerio Paolini

Hi, it's done in version 0.16. Thanks for the patch Regards, Xavier

Sat May 26 03:55:04 2007 x.guimard [...] free.fr - Status changed from 'open' to 'resolved'

Sat May 26 03:55:07 2007 x.guimard [...] free.fr - Given to GUIMARD

Sat May 26 03:59:43 2007 x.guimard [...] free.fr - Severity Normal added

Sat May 26 03:59:43 2007 x.guimard [...] free.fr - Broken in 0.14 added

Mon Mar 17 10:51:25 2008 x.guimard [...] free.fr - Fixed in 0.16 added