Bug #120621 for Net-FTPSSL: Support for IO::Handle objects in 2nd parameter of get() and xget()

Tue Mar 14 11:29:06 2017 cmanley [...] cpan.org - Ticket created

Subject:

Support for IO::Handle objects in 2nd parameter of get() and xget()

The Net::FTP module supports IO::Handle objects as 2nd parameter in the get() method, but the Net::FTPSSL doesn't. This is because Net::FTPSSL checks for a ref and if the ref is a GLOB. The latter check isn't necessary, and if commented out, then Net::FTPSSL works fine. You could choose to either comment the GLOB check out or add an ISA IO::Handle check as well. The IO::Handle objects I use are IO::Compress::Gzip to save to gzipped files. I suspect that the same issue exists when using DebugLogFile but I haven't tried that yet. Thanks

Tue Mar 14 19:37:28 2017 CLEACH [...] cpan.org - Correspondence added

Hi Craig, Sounds like a good enhancement. I'll look into it for the get/put methods. Maybe even for when specifying DebugLogFile after I think about it a bit. Curtis On Tue Mar 14 11:29:06 2017, CMANLEY wrote: Show quoted text

> The Net::FTP module supports IO::Handle objects as 2nd parameter in > the get() method, but the Net::FTPSSL doesn't. This is because > Net::FTPSSL checks for a ref and if the ref is a GLOB. The latter > check isn't necessary, and if commented out, then Net::FTPSSL works > fine. You could choose to either comment the GLOB check out or add an > ISA IO::Handle check as well. > The IO::Handle objects I use are IO::Compress::Gzip to save to gzipped > files. > > I suspect that the same issue exists when using DebugLogFile but I > haven't tried that yet. > > Thanks

Tue Mar 14 19:37:29 2017 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 14 20:00:09 2017 CLEACH [...] cpan.org - Severity Wishlist added

Wed Mar 22 18:14:25 2017 CLEACH [...] cpan.org - Correspondence added

Hi Craig, Your request has been implemented. I just uploaded v0.36. It should be ready to download within the hour. Curtis On Tue Mar 14 19:37:28 2017, CLEACH wrote: Show quoted text

> Hi Craig, > > Sounds like a good enhancement. I'll look into it for the get/put > methods. Maybe even for when specifying DebugLogFile after I think > about it a bit. > > Curtis > > On Tue Mar 14 11:29:06 2017, CMANLEY wrote:

> > The Net::FTP module supports IO::Handle objects as 2nd parameter in > > the get() method, but the Net::FTPSSL doesn't. This is because > > Net::FTPSSL checks for a ref and if the ref is a GLOB. The latter > > check isn't necessary, and if commented out, then Net::FTPSSL works > > fine. You could choose to either comment the GLOB check out or add an > > ISA IO::Handle check as well. > > The IO::Handle objects I use are IO::Compress::Gzip to save to > > gzipped > > files. > > > > I suspect that the same issue exists when using DebugLogFile but I > > haven't tried that yet. > > > > Thanks

Fri Mar 24 12:58:03 2017 CLEACH [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Mar 24 12:58:04 2017 CLEACH [...] cpan.org - Fixed in 0.36 added