From: Ivan Kohler <ivan@freeside.biz>
Date: Tue, 7 Mar 2017 05:33:37 +0000 (-0800)
Subject: detect errors on the scp portion of the backup, RT#74914
X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=da5b18b44fd8f953832a9985602cdd69e81bafb0

detect errors on the scp portion of the backup, RT#74914
---

diff --git a/FS/FS/Cron/backup.pm b/FS/FS/Cron/backup.pm
index cfc8e3624..6e172ae4c 100644
--- a/FS/FS/Cron/backup.pm
+++ b/FS/FS/Cron/backup.pm
@@ -58,7 +58,9 @@ sub backup {
   if ( $scpdest ) {
     eval "use Net::SCP qw(scp);";
     backup_email_and_die($conf,$filename,$@) if $@;
-    scp("/var/tmp/$database.$ext", "$scpdest/$filename.$ext");
+    my $scp = new Net::SCP;
+    $scp->scp("/var/tmp/$database.$ext", "$scpdest/$filename.$ext")
+      or backup_email_and_die($conf, $filename, $scp->{errstr});
   }
 
   unlink "/var/tmp/$database.$ext" or backup_email_and_die($conf,$filename,$!); #or just warn?