Skip to content

Commit 055a709

Browse files
Document intentional spacing in SSH format strings (#96)
* Initial plan * Add clarifying comments for SSH format string spacing Co-authored-by: MichaelKoster70 <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: MichaelKoster70 <[email protected]>
1 parent 0e3ba66 commit 055a709

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/Extension/RemoteDebuggerLauncher/PackageConstants.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ public static class SecureShell
181181
public const string DefaultKnownHostsFileName = "known_hosts";
182182

183183
/// <summary>SSH key scanner arguments.</summary>
184+
/// <remarks>No space after {2} because SshForceIPv4 includes a trailing space.</remarks>
184185
public const string KeyScanArguments = "{2}-p {1} {0}";
185186

186187
/// <summary>Argument for SSH to force IPv4 usage.</summary>
@@ -190,6 +191,7 @@ public static class SecureShell
190191
public const string SshExecutable = "ssh.exe";
191192

192193
/// <summary>The SSH arguments to add server fingerprint to known_hosts file.</summary>
194+
/// <remarks>No space after {4} because SshForceIPv4 includes a trailing space.</remarks>
193195
public const string SshArguments = "{0}@{1} {4}-p {2} -i \"{3}\" \"echo DONE\"";
194196

195197
/// <summary>HTTPS Developer Certificate name.</summary>

0 commit comments

Comments
 (0)