bug: added escaping the cmd arguments for log-path

ibetitsmike · ibetitsmike · commit a9a1cb264285 · 2025-06-17T17:48:02.000Z
diff --git a/src/remote.ts b/src/remote.ts
@@ -677,7 +677,7 @@ export class Remote {
 		this.storage.writeToCoderOutputChannel(
 			`SSH proxy diagnostics are being written to ${logDir}`,
 		);
-		return ` --log-dir ${escape(logDir)}`;
+		return ` --log-dir ${escapeCommandArg(logDir)}`;
 	}
 
 	// updateSSHConfig updates the SSH configuration with a wildcard that handles

Original file line number	Diff line number	Diff line change
`@@ -677,7 +677,7 @@ export class Remote {`
`677`	`677`	`this.storage.writeToCoderOutputChannel(`
`678`	`678`	`SSH proxy diagnostics are being written to ${logDir}`,
`679`	`679`	`);`
`680`		- return ` --log-dir ${escape(logDir)}`;
	`680`	+ return ` --log-dir ${escapeCommandArg(logDir)}`;
`681`	`681`	`}`
`682`	`682`
`683`	`683`	`// updateSSHConfig updates the SSH configuration with a wildcard that handles`