[Issue #366] escape paths in restore_command with double quotes when running restore

Author: gsmolk

src/restore.c

@@ -1375,7 +1375,7 @@ print_recovery_settings(FILE *fp, pgBackup *backup,
 	else
 	{
 		/* default cmdline, ok for local restore */
-		sprintf(restore_command_guc, "%s archive-get -B %s --instance %s "
+		sprintf(restore_command_guc, "\"%s\" archive-get -B \"%s\" --instance \"%s\" "
 				"--wal-file-path=%%p --wal-file-name=%%f",
 				PROGRAM_FULL_PATH ? PROGRAM_FULL_PATH : PROGRAM_NAME,
 				backup_path, instance_name);

tests/archive.py

@@ -1715,7 +1715,7 @@ def test_archive_options(self):
             recovery_content = f.read()
 
         self.assertIn(
-            "restore_command = '{0} archive-get -B {1} --instance {2} "
+            "restore_command = '\"{0}\" archive-get -B \"{1}\" --instance \"{2}\" "
             "--wal-file-path=%p --wal-file-name=%f --remote-host=localhost "
             "--remote-port=22 --remote-user={3}'".format(
                 self.probackup_path, backup_dir, 'node', self.user),
@@ -1782,7 +1782,7 @@ def test_archive_options_1(self):
         self.restore_node(
             backup_dir, 'node', node,
             options=[
-                '--restore-command=none'.format(wal_dir),
+                '--restore-command=none',
                 '--archive-host=localhost1',
                 '--archive-port=23',
                 '--archive-user={0}'.format(self.user)
@@ -1792,7 +1792,7 @@ def test_archive_options_1(self):
             recovery_content = f.read()
 
         self.assertIn(
-            "restore_command = '{0} archive-get -B {1} --instance {2} "
+            "restore_command = '\"{0}\" archive-get -B \"{1}\" --instance \"{2}\" "
             "--wal-file-path=%p --wal-file-name=%f --remote-host=localhost1 "
             "--remote-port=23 --remote-user={3}'".format(
                 self.probackup_path, backup_dir, 'node', self.user),