Upgrade vite to 6.0.11

[avatus]

This will upgrade vite. I'm getting an "unmet dep" warning for electron-vite saying it expects a max of vite 5, however we are currently on the latest version of electron-vite so I don't think there is much we can do here. We have already been running vite 6+ for months now tho and electron-vite has been working fine so I think its ok to ignore for now.

@ryanclark , i didn't have any issues with localhost and my non-localhost domain that I typically run so I didn't need to update allowedHosts, can you confirm?

edit: confirmed we had a few users whos HMR didnt work without allowedHosts so Ive updated it

[avatus]

Waiting for gravitational/shared-workflows#310 before I can merge

[ravicious]

I wonder if we could use the target variable instead of allowing all hosts?

I confirmed that HMR doesn't work when I comment out allowedHosts completely and that it still works with this solution in place.

diff --git a/web/packages/build/vite/config.ts b/web/packages/build/vite/config.ts
index e2e9db9f1d..9521631ddf 100644
--- a/web/packages/build/vite/config.ts
+++ b/web/packages/build/vite/config.ts
@@ -54,10 +54,15 @@ export function createViteConfig(
       }
     }
 
+    const targetHostname =
+      target !== DEFAULT_PROXY_TARGET
+        ? new URL(`http://${target}`).hostname
+        : undefined;
+
     const config: UserConfig = {
       clearScreen: false,
       server: {
-        allowedHosts: true,
+        allowedHosts: targetHostname ? [`.${targetHostname}`] : [],
         fs: {
           allow: [rootDirectory, '.'],
         },

[avatus]

patch works for me, i appreciate the improvement. applied

[ravicious]

Vite was updated to v6 quite recently actually. We made the decision to update it to v6 without waiting for electron-vite. #50617 (review)

[ravicious]

BTW, I just got a notification an hour ago under an electron-vite issue that the beta of v3 is out which updates Vite compatibility to v6.