Node Explorer: Fix issue with pre-existing config block (#276)

tylersmalley · web-flow · commit de645ae3452b · 2023-11-21T08:58:59.000-08:00
Closes #270 Signed-off-by: Tyler Smalley <tyler@tailscale.com>
diff --git a/src/node-explorer-provider.ts b/src/node-explorer-provider.ts
@@ -703,7 +703,7 @@ export class NodeExplorerProvider
       async (node: PeerRoot | FileExplorer) => {
         const { addr, path } = extractAddrAndPath(node);
 
-        if (addr && this.configManager.config.hosts?.[addr].persistToSSHConfig !== false) {
+        if (addr && this.configManager.config.hosts?.[addr]?.persistToSSHConfig !== false) {
           await syncSSHConfig(addr, this.configManager);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -703,7 +703,7 @@ export class NodeExplorerProvider`
`703`	`703`	`async (node: PeerRoot \| FileExplorer) => {`
`704`	`704`	`const { addr, path } = extractAddrAndPath(node);`
`705`	`705`
`706`		`- if (addr && this.configManager.config.hosts?.[addr].persistToSSHConfig !== false) {`
	`706`	`+ if (addr && this.configManager.config.hosts?.[addr]?.persistToSSHConfig !== false) {`
`707`	`707`	`await syncSSHConfig(addr, this.configManager);`
`708`	`708`	`}`
`709`	`709`