From cbde64bccd3d2693ba1597ee0fcce007cadbd524 Mon Sep 17 00:00:00 2001
From: Daniel Falbel <dfalbel@gmail.com>
Date: Fri, 21 Feb 2025 13:13:22 -0300
Subject: [PATCH] Don't add URL for unavailable icons

---
 .../browser/positronConnectionsInstance.ts             | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/vs/workbench/services/positronConnections/browser/positronConnectionsInstance.ts b/src/vs/workbench/services/positronConnections/browser/positronConnectionsInstance.ts
index ed987fc208a..ce24d5f3d9a 100644
--- a/src/vs/workbench/services/positronConnections/browser/positronConnectionsInstance.ts
+++ b/src/vs/workbench/services/positronConnections/browser/positronConnectionsInstance.ts
@@ -75,7 +75,13 @@ export class PositronConnectionsInstance extends BaseConnectionsInstance impleme
 			try {
 				// Failing to acquire the icon is fine
 				// We just log the error
-				object.metadata.icon = FileAccess.uriToBrowserUri(URI.file(await object.getIcon())).toString();
+				let icon: string | undefined = await object.getIcon();
+				if (!icon || icon === '') {
+					icon = undefined;
+				} else {
+					icon = FileAccess.uriToBrowserUri(URI.file(icon)).toString();
+				}
+				object.metadata.icon = icon;
 			} catch (err: any) {
 				service.log(`Failed to get icon for ${object.id}: ${err.message}`);
 			}
@@ -461,7 +467,7 @@ class PositronConnectionItem implements IPositronConnectionItem {
 
 	private async getIcon() {
 		const icon = await this.instance.client.getIcon(this.path);
-		if (icon === '') {
+		if (!icon || icon === '') {
 			return undefined;
 		} else {
 			return FileAccess.uriToBrowserUri(URI.file(icon)).toString();