Bug #53522 for Net-OpenSSH: Net::OpenSSH remove _check_is_system

Fri Jan 08 14:23:28 2010 horne [...] infoblox.com - Ticket created

Subject:	Net::OpenSSH remove _check_is_system_fh requirement
Date:	Fri, 08 Jan 2010 11:22:39 -0800
To:	bug-Net-OpenSSH [...] rt.cpan.org
From:	Geoff Horne <horne [...] infoblox.com>

i'm trying to use scp_put(), and if there is a permission error on the remote side the error goes to STDERR. i want to capture that error, or at least suppress it since this is running inside a CGI/mod_perl all. perl 5.8 lets you redirect filehandles to a variable thus : open(STDERR, '>', \$errstring); but fileno() doesn't return a valid numnber : (Filehandles connected to memory objects via new features of "open" may return undefined even though they are open.) so i'm stuck, the only workaround is to redirect the output to /dev/null

Fri Jan 08 16:19:56 2010 sfandino [...] yahoo.com - Correspondence added

Subject:	Re: [rt.cpan.org #53522] Net::OpenSSH remove _check_is_system_fh requirement
Date:	Fri, 8 Jan 2010 13:19:38 -0800 (PST)
To:	bug-Net-OpenSSH [...] rt.cpan.org
From:	Salvador Fandino <sfandino [...] yahoo.com>

Hi, Show quoted text

> i'm trying to use scp_put(), and if there is a permission error on the

Show quoted text

> remote side the error goes to STDERR. > > i want to capture that error, or at least suppress it since this is running > inside a CGI/mod_perl all. > > perl 5.8 lets you redirect filehandles to a variable thus : > > open(STDERR, '>', \$errstring); > > but fileno() doesn't return a valid numnber : > > (Filehandles connected to memory objects via new features of > "open" may return undefined even though they are open.) > > so i'm stuck, the only workaround is to redirect the output to /dev/null

The error messages come from the scp process forked by Net::OpenSSH. Unfortunately, the perl redirection to a scalar feature does not work across forks and actually, that's the reason the _check_is_system_fh check is there. Removing the test will not solve the problem! Besides sending stderr to /dev/null, you can also send it to a temporal file or to a log so it doesn't get lost. Anyway, scp is called by default with the quiet flag set, so no errors errors should be printed to stderr. Can you send me the exact error message you are getting? - Salva

Fri Jan 08 16:19:57 2010 The RT System itself - Status changed from 'new' to 'open'

Fri Jan 08 16:25:35 2010 salva [...] cpan.org - TimeWorked changed from (no value) to '10'

Fri Jan 08 16:25:35 2010 salva [...] cpan.org - Status changed from 'open' to 'rejected'

Fri Jan 08 16:44:40 2010 horne [...] infoblox.com - Correspondence added

Subject:	Re: [rt.cpan.org #53522] Net::OpenSSH remove _check_is_system_fh requirement
Date:	Fri, 08 Jan 2010 13:44:24 -0800
To:	bug-Net-OpenSSH [...] rt.cpan.org
From:	Geoff Horne <horne [...] infoblox.com>

On 1/8/10 1:19 PM, Salvador \"Fandiño\" via RT wrote: Show quoted text

> Removing the test will not solve the problem!

understood. Show quoted text

> Anyway, scp is called by default with the quiet flag set, so no errors > errors should be printed to stderr. Can you send me the exact error > message you are getting?

try and scp a file to a directory that doesn't exist, you get the following : scp: /var/named/include/named-include.conf: No such file or directory

Fri Jan 08 16:44:41 2010 The RT System itself - Status changed from 'rejected' to 'open'

Mon Apr 19 06:13:51 2010 salva [...] cpan.org - Correspondence added 60 min

scp quiet option (-q) does not suppress errors, only warnings and diagnostics. Silence the error as follows: $ssh->scp_put({stderr_discard => 1}, ...); $ssh->error and warn "scp operation failed: " . $ssh->error;

Mon Apr 19 06:13:51 2010 salva [...] cpan.org - Status changed from 'open' to 'rejected'

Mon Apr 19 06:13:53 2010 salva [...] cpan.org - Given to SALVA

Bug #53522 for Net-OpenSSH: Net::OpenSSH remove _check_is_system_fh requirement