Bug #99454 for Net-SMTP-SSL: Fails with Net::SMTP 2.35 and later

Sun Oct 12 23:05:34 2014 ANDK [...] cpan.org - Ticket created

CC:	SHAY [...] cpan.org
Subject:	Fails with Net::SMTP 2.35 and later

Sample fail report: http://www.cpantesters.org/cpan/report/46517962 Downgrading to SHAY/libnet-1.27.tar.gz (Net::SMTP 2.34) made tests pass again for me. Let me know if you need further detail. HTH&&Thanks, (Disclaimer: this was discovered by statistical analysis, I'm not speaking as a user)

Tue Oct 28 13:48:01 2014 ether [...] cpan.org - Correspondence added

RT-Send-CC:

shay [...] cpan.org

On 2014-10-12 20:05:34, ANDK wrote: Show quoted text

> Sample fail report: > > http://www.cpantesters.org/cpan/report/46517962 > > Downgrading to SHAY/libnet-1.27.tar.gz (Net::SMTP 2.34) made tests > pass again for me. Let me know if you need further detail. > > HTH&&Thanks, > (Disclaimer: this was discovered by statistical analysis, I'm not > speaking as a user)

I also experience this issue: Not a GLOB reference at .../Net/SMTP/SSL.pm line 16. This code looks pretty dodgy: no strict 'refs'; foreach ( keys %Net::SMTP:: ) { next unless defined *{$Net::SMTP::{$_}}{CODE}; *{$_} = \&{"Net::SMTP::$_"}; } I would recommend using a wrapper instead to manipulate your symbol table, e.g. Package::Stash.

Tue Oct 28 13:48:02 2014 The RT System itself - Status changed from 'new' to 'open'

Tue Oct 28 13:48:19 2014 ether [...] cpan.org - Severity Critical added

Tue Oct 28 13:48:41 2014 ether [...] cpan.org - Requestor ETHER added

Wed Oct 29 04:12:50 2014 shay [...] cpan.org - Correspondence added

Perhaps Net::SMTP::SSL->new() should just create a standard Net::SMTP object with the appropriate arguments for an SSL connection (i.e. Net::SMTP->new(SSL => 1)) if it detects that a recent Net::SMTP is installed: Net::SMTP now supports SSL connections itself as of version 2.35 (in libnet-1.28).

Wed Oct 29 12:48:11 2014 ether [...] cpan.org - Correspondence added

On 2014-10-29 01:12:50, SHAY wrote: Show quoted text

> Perhaps Net::SMTP::SSL->new() should just create a standard Net::SMTP > object with the appropriate arguments for an SSL connection (i.e. > Net::SMTP->new(SSL => 1)) if it detects that a recent Net::SMTP is > installed: Net::SMTP now supports SSL connections itself as of version > 2.35 (in libnet-1.28).

That sounds ideal. Net::SMTP::SSL hasn't been updated since 2004... is the author still around? Perhaps this module should be transferred into the Net-SMTP distribution (PAUSE permissions would need to be adjusted).

Mon Dec 08 19:06:21 2014 ether [...] cpan.org - Correspondence added

On 2014-10-29 09:48:11, ETHER wrote: Show quoted text

> On 2014-10-29 01:12:50, SHAY wrote:

> > Perhaps Net::SMTP::SSL->new() should just create a standard Net::SMTP > > object with the appropriate arguments for an SSL connection (i.e. > > Net::SMTP->new(SSL => 1)) if it detects that a recent Net::SMTP is > > installed: Net::SMTP now supports SSL connections itself as of > > version > > 2.35 (in libnet-1.28).

> > That sounds ideal. > > Net::SMTP::SSL hasn't been updated since 2004... is the author still > around? Perhaps > this module should be transferred into the Net-SMTP distribution > (PAUSE permissions > would need to be adjusted).

Paging Casey West -- are you receptive to folding this distribution into Net-SMTP, or at least turning it over to SHAY?

Thu Mar 19 13:50:17 2015 zefram [...] fysh.org - Correspondence added

Subject:	Re: [rt.cpan.org #99454] Fails with Net::SMTP 2.35 and later
Date:	Thu, 19 Mar 2015 17:50:04 +0000
To:	bug-Net-SMTP-SSL [...] rt.cpan.org
From:	Zefram <zefram [...] fysh.org>

The attached patch is a straight fix for what Net-SMTP-SSL is trying to do. -zefram

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

Thu Mar 26 18:08:39 2015 rjbs [...] cpan.org - Correspondence added

Thanks, patch applied and released. (How did I become first-come on this? Woe is me!) -- rjbs

Thu Mar 26 18:08:46 2015 rjbs [...] cpan.org - Status changed from 'open' to 'resolved'

Tue May 05 20:09:26 2015 ether [...] cpan.org - Fixed in 1.02 added