fix: prevent duplicate include entries in SSH config (#339)

hakhandelwal11 · web-flow · commit ed9c8a211874 · 2026-03-24T09:14:53.000-07:00
diff --git a/pkg/ssh/sshconfigurer.go b/pkg/ssh/sshconfigurer.go
@@ -611,7 +611,13 @@ func makeIncludeBrevStr(brevSSHConfigPath string) string {
 }
 
 func doesUserSSHConfigIncludeBrevConfig(conf string, brevConfigPath string) bool {
-	return strings.Contains(conf, makeIncludeBrevStr(brevConfigPath))
+	if strings.Contains(conf, makeIncludeBrevStr(brevConfigPath)) {
+		return true
+	}
+	if strings.Contains(conf, makeIncludeBrevStr(toWindowsPath(brevConfigPath))) {
+		return true
+	}
+	return false
 }
 
 // Deprecated: var _ Config = SSHConfigurerServiceMesh{}

Original file line number	Diff line number	Diff line change
`@@ -611,7 +611,13 @@ func makeIncludeBrevStr(brevSSHConfigPath string) string {`
`611`	`611`	`}`
`612`	`612`
`613`	`613`	`func doesUserSSHConfigIncludeBrevConfig(conf string, brevConfigPath string) bool {`
`614`		`- return strings.Contains(conf, makeIncludeBrevStr(brevConfigPath))`
	`614`	`+ if strings.Contains(conf, makeIncludeBrevStr(brevConfigPath)) {`
	`615`	`+ return true`
	`616`	`+ }`
	`617`	`+ if strings.Contains(conf, makeIncludeBrevStr(toWindowsPath(brevConfigPath))) {`
	`618`	`+ return true`
	`619`	`+ }`
	`620`	`+ return false`
`615`	`621`	`}`
`616`	`622`
`617`	`623`	`// Deprecated: var _ Config = SSHConfigurerServiceMesh{}`