potyarkin/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

Put host last in SSH command line

Browse source 
This is how the SSH manpage says the command line should look, and the
"--" prevents mistakes in hostnames from being interpreted as options.
This commit is contained in:
greatroar 2020-02-19 15:53:20 +01:00
parent 6ac6bca7a1
commit e1969d1e33
2 changed files with 10 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

12
internal/backend/sftp/sshcmd_test.go
View file

@ -13,34 +13,34 @@ var sshcmdTests = []struct {
{
Config{User: "user", Host: "host", Path: "dir/subdir"},
"ssh",
[]string{"host", "-l", "user", "-s", "sftp"},
[]string{"-l", "user", "-s", "sftp", "--", "host"},
},
{
Config{Host: "host", Path: "dir/subdir"},
"ssh",
[]string{"host", "-s", "sftp"},
[]string{"-s", "sftp", "--", "host"},
},
{
Config{Host: "host", Port: "10022", Path: "/dir/subdir"},
"ssh",
[]string{"host", "-p", "10022", "-s", "sftp"},
[]string{"-p", "10022", "-s", "sftp", "--", "host"},
},
{
Config{User: "user", Host: "host", Port: "10022", Path: "/dir/subdir"},
"ssh",
[]string{"host", "-p", "10022", "-l", "user", "-s", "sftp"},
[]string{"-p", "10022", "-l", "user", "-s", "sftp", "--", "host"},
},
{
// IPv6 address.
Config{User: "user", Host: "::1", Path: "dir"},
"ssh",
[]string{"::1", "-l", "user", "-s", "sftp"},
[]string{"-l", "user", "-s", "sftp", "--", "::1"},
},
{
// IPv6 address with zone and port.
Config{User: "user", Host: "::1%lo0", Port: "22", Path: "dir"},
"ssh",
[]string{"::1%lo0", "-p", "22", "-l", "user", "-s", "sftp"},
[]string{"-p", "22", "-l", "user", "-s", "sftp", "--", "::1%lo0"},
},
}