},
'groups_susp_reason' => { label =>
'Radius group mapping to reason (via template user)',
- type => 'textarea',
- },
- 'ignore_all_output' => {
- label => 'Ignore all output and errors from the command',
+ type => 'textarea',
+ },
+ 'fail_on_output' => {
+ label => 'Treat any output from the command as an error',
+ type => 'checkbox',
+ },
+ 'ignore_all_errors' => {
+ label => 'Ignore all errors from the command',
type => 'checkbox',
},
'ignored_errors' => { label => 'Regexes of specific errors to ignore, separated by newlines',
$locale = shell_quote $locale;
my $command_string = eval(qq("$command"));
+
my @ssh_cmd_args = (
user => $self->option('user') || 'root',
host => $self->machine,
command => $command_string,
stdin_string => $stdin_string,
- ignore_all_output => $self->option('ignore_all_output'),
ignored_errors => $self->option('ignored_errors') || '',
- );
+ ignore_all_errors => $self->option('ignore_all_errors'),
+ fail_on_output => $self->option('fail_on_output'),
+ );
if($self->option($action . '_no_queue')) {
# discard return value just like freeside-queued.
eval { ssh_cmd(@ssh_cmd_args) };
$error = $@;
+ $error = $error->full_message if ref $error; # Exception::Class::Base
return $error. ' ('. $self->exporttype. ' to '. $self->machine. ')'
if $error;
}
sub _export_replace {
my($self, $new, $old ) = (shift, shift, shift);
my $command = $self->option('usermod');
+ return '' if $command =~ /^\s*$/;
my $stdin = $self->option('usermod_stdin');
no strict 'vars';
{
host => $self->machine,
command => $command_string,
stdin_string => $stdin_string,
- ignore_all_output => $self->option('ignore_all_output'),
- ignored_errors => $self->option('ignored_errors') || '',
+ ignored_errors => $self->option('ignored_errors') || '',
+ ignore_all_errors => $self->option('ignore_all_errors'),
+ fail_on_output => $self->option('fail_on_output'),
);
if($self->option('usermod_no_queue')) {
# discard return value just like freeside-queued.
eval { ssh_cmd(@ssh_cmd_args) };
$error = $@;
+ $error = $error->full_message if ref $error; # Exception::Class::Base
return $error. ' ('. $self->exporttype. ' to '. $self->machine. ')'
if $error;
}
$opt->{'user'}.'@'.$opt->{'host'},
'default_stdin_fh' => $def_in
);
- # ignore_all_output doesn't override this
+ # ignore_all_errors doesn't override SSH connection/auth errors--
+ # probably correct
die "Couldn't establish SSH connection: ". $ssh->error if $ssh->error;
my $ssh_opt = {};
$ssh_opt->{'stdin_data'} = $opt->{'stdin_string'}
if exists($opt->{'stdin_string'}) and length($opt->{'stdin_string'});
+
my ($output, $errput) = $ssh->capture2($ssh_opt, $opt->{'command'});
- return if $opt->{'ignore_all_output'};
+
+ return if $opt->{'ignore_all_errors'};
die "Error running SSH command: ". $ssh->error if $ssh->error;
- if ($errput && $opt->{'ignored_errors'} && length($opt->{'ignored_errors'})) {
+ if ( ($output || $errput)
+ && $opt->{'ignored_errors'} && length($opt->{'ignored_errors'})
+ ) {
my @ignored_errors = split('\n',$opt->{'ignored_errors'});
foreach my $ignored_error ( @ignored_errors ) {
+ $output =~ s/$ignored_error//g;
$errput =~ s/$ignored_error//g;
}
- chomp($errput);
+ $output =~ s/[\s\n]//g;
+ $errput =~ s/[\s\n]//g;
}
- die $errput if $errput;
- die $output if $output;
+
+ die "$errput\n" if $errput;
+ die "$output\n" if $output and $opt->{'fail_on_output'};
'';
}
#sub shellcommands_delete { #subroutine, not method
#}
+sub _upgrade_exporttype {
+ my $class = shift;
+ $class =~ /^FS::part_export::(\w+)$/;
+ foreach my $self ( qsearch('part_export', { 'exporttype' => $1 }) ) {
+ my %options = $self->options;
+ my $changed = 0;
+ # 2011-12-13 - 2012-02-16: ignore_all_output option
+ if ( $options{'ignore_all_output'} ) {
+ # ignoring STDOUT is now the default
+ $options{'ignore_all_errors'} = 1;
+ delete $options{'ignore_all_output'};
+ $changed++;
+ }
+ my $error = $self->replace(%options) if $changed;
+ die $error if $error;
+ }
+}
+
1;