X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=scpubgit%2FObject-Remote.git;a=blobdiff_plain;f=lib%2FObject%2FRemote%2FConnector%2FSSH.pm;h=dbc988a906b798f706a96b892724cedfb28683b4;hp=363e214d7156575d42c78ee22a3f0d89f625fda1;hb=8506bc0849fa561ef232a09e0394c417bc437aa2;hpb=f129bfaf05b1ae0e2e2992cad47a70482dec9885 diff --git a/lib/Object/Remote/Connector/SSH.pm b/lib/Object/Remote/Connector/SSH.pm index 363e214..dbc988a 100644 --- a/lib/Object/Remote/Connector/SSH.pm +++ b/lib/Object/Remote/Connector/SSH.pm @@ -2,6 +2,7 @@ package Object::Remote::Connector::SSH; use Object::Remote::ModuleSender; use Object::Remote::Handle; +use String::ShellQuote; use Moo; with 'Object::Remote::Role::Connector::PerlInterpreter'; @@ -14,24 +15,14 @@ has ssh_options => (is => 'ro', default => sub { [ '-A' ] }); has ssh_command => (is => 'ro', default => sub { 'ssh' }); -sub _escape_shell_arg { - my ($self, $str) = (@_); - $str =~ s/((?:^|[^\\])(?:\\\\)*)'/$1\\'/g; - return "$str"; -} - - sub _build_ssh_perl_command { my ($self) = @_; - my $perl_command = join('', @{$self->perl_command}); - - #TODO non-trivial to escape properly for ssh and shell - #this "works" but is not right, needs to be replaced - #after testing + my $perl_command = $self->perl_command; + return [ do { my $c = $self->ssh_command; ref($c) ? @$c : $c }, @{$self->ssh_options}, $self->ssh_to, - $self->_escape_shell_arg($perl_command), + shell_quote(@$perl_command), ]; }