Bug #107307 for Email-Send-SMTP-Gmail: Large message body also needs to be split into chunks for successfull sending

Thu Sep 24 17:53:13 2015 v0rd [...] yandex.ru - Ticket created

Subject:	Large message body also needs to be split into chunks for successfull sending
Date:	Fri, 25 Sep 2015 00:52:59 +0300
To:	bug-email-send-smtp-gmail [...] rt.cpan.org
From:	Воробьев Дмитрий <v0rd [...] yandex.ru>

Hi. I've encountered the same problem described in https://rt.cpan.org/Public/Bug/Display.html?id=101832 in case of large message body. According to the source message body is sent "as is" without chunking in 2 places in code:

$self->{sender}->datasend($mail->{body} . "\n\n");

And this will lead to drop of the connection. I've changed these lines to:

my @groups_body = split(/(.{76})/,$mail->{body});
$self->{sender}->datasend($_) foreach @groups_body;
$self->{sender}->datasend("\n\n");

as for the code sending atthachments and this has fixed the problem.

Maybe this should be included in th next version.

Thanks,

Dmitry

Sun Sep 27 07:00:12 2015 PECO [...] cpan.org - Correspondence added

Thank you very much. It will be added into the next release! Regards On Thu Sep 24 17:53:13 2015, v0rd@yandex.ru wrote: Show quoted text

> Hi. I've encountered the same problem described in > https://rt.cpan.org/Public/Bug/Display.html?id=101832 in case of large > message > body. According to the source message body is sent "as is" without > chunking in > 2 places in code: $self->{sender}->datasend($mail->{body} . "\n\n"); > And this > will lead to drop of the connection. I've changed these lines to: my > @groups_body = split(/(.{76})/,$mail->{body}); > $self->{sender}->datasend($_) foreach @groups_body; > $self->{sender}->datasend("\n\n"); as for the code sending > atthachments and > this has fixed the problem.Maybe this should be included in th next > version. > Thanks,Dmitry

Sun Sep 27 07:00:13 2015 The RT System itself - Status changed from 'new' to 'open'

Sun Sep 27 07:00:15 2015 PECO [...] cpan.org - Status changed from 'open' to 'patched'

Mon Jun 26 07:23:08 2017 PECO [...] cpan.org - Correspondence added 1 min

Resolved! On Sun Sep 27 07:00:12 2015, PECO wrote: Show quoted text

> Thank you very much. > > It will be added into the next release! > > Regards > > On Thu Sep 24 17:53:13 2015, v0rd@yandex.ru wrote:

> > Hi. I've encountered the same problem described in > > https://rt.cpan.org/Public/Bug/Display.html?id=101832 in case of large > > message > > body. According to the source message body is sent "as is" without > > chunking in > > 2 places in code: $self->{sender}->datasend($mail->{body} . "\n\n"); > > And this > > will lead to drop of the connection. I've changed these lines to: my > > @groups_body = split(/(.{76})/,$mail->{body}); > > $self->{sender}->datasend($_) foreach @groups_body; > > $self->{sender}->datasend("\n\n"); as for the code sending > > atthachments and > > this has fixed the problem.Maybe this should be included in th next > > version. > > Thanks,Dmitry

> >

Mon Jun 26 09:32:32 2017 PECO [...] cpan.org - Correspondence added

Bug-report closed! Thank you very much Peco On Mon Jun 26 07:23:08 2017, PECO wrote: Show quoted text

> Resolved! > > On Sun Sep 27 07:00:12 2015, PECO wrote:

> > Thank you very much. > > > > It will be added into the next release! > > > > Regards > > > > On Thu Sep 24 17:53:13 2015, v0rd@yandex.ru wrote:

> > > Hi. I've encountered the same problem described in > > > https://rt.cpan.org/Public/Bug/Display.html?id=101832 in case of large > > > message > > > body. According to the source message body is sent "as is" without > > > chunking in > > > 2 places in code: $self->{sender}->datasend($mail->{body} . "\n\n"); > > > And this > > > will lead to drop of the connection. I've changed these lines to: my > > > @groups_body = split(/(.{76})/,$mail->{body}); > > > $self->{sender}->datasend($_) foreach @groups_body; > > > $self->{sender}->datasend("\n\n"); as for the code sending > > > atthachments and > > > this has fixed the problem.Maybe this should be included in th next > > > version. > > > Thanks,Dmitry

> > > >

> >

Mon Jun 26 09:32:33 2017 PECO [...] cpan.org - Status changed from 'patched' to 'resolved'

Mon Jun 26 09:32:38 2017 PECO [...] cpan.org - Taken

Mon Jun 26 09:32:38 2017 PECO [...] cpan.org - Fixed in 1.21 added