From 5c4b6e7b6fdeb569cd4ca0bf28ea2e75faf68ecf Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@windmill.dev>
Date: Thu, 10 Jul 2025 13:07:05 +0000
Subject: [PATCH] nits

---
 backend/windmill-worker/src/worker.rs           | 1 +
 frontend/src/lib/components/instanceSettings.ts | 5 ++---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/backend/windmill-worker/src/worker.rs b/backend/windmill-worker/src/worker.rs
index 20d063dfc9..1278624c9a 100644
--- a/backend/windmill-worker/src/worker.rs
+++ b/backend/windmill-worker/src/worker.rs
@@ -1907,6 +1907,7 @@ pub async fn run_worker(
             tracing::error!("error in awaiting send_result process: {e:?}")
         }
     }
+    tracing::info!(worker = %worker_name, hostname = %hostname, "waiting for interactive_shell to finish");
     if let Some(interactive_shell) = interactive_shell {
         if let Err(e) = interactive_shell.await {
             tracing::error!("error in awaiting interactive_shell process: {e:?}")
diff --git a/frontend/src/lib/components/instanceSettings.ts b/frontend/src/lib/components/instanceSettings.ts
index cc18716488..4f278ec7e8 100644
--- a/frontend/src/lib/components/instanceSettings.ts
+++ b/frontend/src/lib/components/instanceSettings.ts
@@ -82,12 +82,11 @@ export const settings: Record<string, Setting[]> = {
 			storage: 'setting',
 			error: 'Base url must start with http:// or https:// and not end with / or a space',
 			isValid: (value: string | undefined) =>
-				value
-					? value?.startsWith('http') &&
+				value == undefined
+					|| value?.startsWith('http') &&
 						value.includes('://') &&
 						!value?.endsWith('/') &&
 						!value?.endsWith(' ')
-					: false
 		},
 		{
 			label: 'Email domain',