Conversation
#9455) 1. noVNC fix: remove @novnc/novnc and guacamole-common-js from Vite build.rollupOptions.external and optimizeDeps.exclude. They stay in ssr.external (server-side only). Vite now bundles them for the client — fixes "module not available" runtime error. 2. KASM cards: new ReactKasmCards component wired into AstroVMDashboard.astro. Shows KASM workspaces with: - Start/Stop buttons (scale deployment 0↔1) - VPN status indicator (Gluetun sidecar) - Open button for KASM web access - Bitmask state flags matching VM cards pattern
Contributor
Author
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
…9457) Add cross-subdomain auth for *.kbve.com: Shared cookie (kbve_auth_token): - Set on .kbve.com domain after OAuth callback on any subdomain - AuthBridge.handleCallback() sets it, signOut() clears it - chat.kbve.com checks cookie if no IDB session found - 1h TTL matching Supabase JWT expiry Redirect relay (/auth/relay?token=jwt): - Accepts JWT from URL param (for "Chat" links on kbve.com) - Validates JWT structure, stores in shared cookie - Cleans token from URL history, redirects to /chat Auth check order on chat.kbve.com: 1. IDB session (same-origin, from direct OAuth on chat.kbve.com) 2. Shared cookie (cross-origin, from OAuth on kbve.com) 3. Show login prompt (independent OAuth as fallback)
…9458) The staff_permissions RPC resolves after the initial auth session. fetchAll() ran with isStaff=false on first call, marking staff cards (Grafana, ArgoCD, ClickHouse, ROWS) as 'unavailable'. Fix: re-check $auth flags directly in fetchAll() before deciding which services to load. If staff resolved late, update isStaff and fetch staff-only services.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release: Dev → Main
5 atomic commits ready for main
Features
b71f724)Bug Fixes
786f2e1)c490460)Documentation
3564687)Chores
b11be00)This PR is automatically maintained by CI — KBVE Studio