Revert "fix: Avoid double escaping of ProxyCommand on Windows (#3664)" (#3704)

mafredri · web-flow · commit 98a6958f1059 · 2022-08-26T17:52:25.000+03:00
This reverts commit 123fe01.
diff --git a/cli/configssh.go b/cli/configssh.go
@@ -280,16 +280,10 @@ func configSSH() *cobra.Command {
 						"\tLogLevel ERROR",
 					)
 					if !skipProxyCommand {
-						// In SSH configs, strings inside "" are interpreted literally and there
-						// is no need to e.g. escape backslashes (common on Windows platforms).
-						// We will escape the quotes, though.
-						escapedBinaryFile := strings.ReplaceAll(binaryFile, "\"", "\\\"")
 						if !wireguard {
-							//nolint:gocritic // We don't want to use %q here, see above.
-							configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand \"%s\" --global-config \"%s\" ssh --stdio %s", escapedBinaryFile, root, hostname))
+							configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand %q --global-config %q ssh --stdio %s", binaryFile, root, hostname))
 						} else {
-							//nolint:gocritic // We don't want to use %q here, see above.
-							configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand \"%s\" --global-config \"%s\" ssh --wireguard --stdio %s", escapedBinaryFile, root, hostname))
+							configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand %q --global-config %q ssh --wireguard --stdio %s", binaryFile, root, hostname))
 						}
 					}
 

Original file line number	Diff line number	Diff line change
`@@ -280,16 +280,10 @@ func configSSH() *cobra.Command {`
`280`	`280`	`"\tLogLevel ERROR",`
`281`	`281`	`)`
`282`	`282`	`if !skipProxyCommand {`
`283`		`- // In SSH configs, strings inside "" are interpreted literally and there`
`284`		`- // is no need to e.g. escape backslashes (common on Windows platforms).`
`285`		`- // We will escape the quotes, though.`
`286`		`- escapedBinaryFile := strings.ReplaceAll(binaryFile, "\"", "\\\"")`
`287`	`283`	`if !wireguard {`
`288`		`- //nolint:gocritic // We don't want to use %q here, see above.`
`289`		`- configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand \"%s\" --global-config \"%s\" ssh --stdio %s", escapedBinaryFile, root, hostname))`
	`284`	`+ configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand %q --global-config %q ssh --stdio %s", binaryFile, root, hostname))`
`290`	`285`	`} else {`
`291`		`- //nolint:gocritic // We don't want to use %q here, see above.`
`292`		`- configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand \"%s\" --global-config \"%s\" ssh --wireguard --stdio %s", escapedBinaryFile, root, hostname))`
	`286`	`+ configOptions = append(configOptions, fmt.Sprintf("\tProxyCommand %q --global-config %q ssh --wireguard --stdio %s", binaryFile, root, hostname))`
`293`	`287`	`}`
`294`	`288`	`}`
`295`	`289`