Skip to content

feat: Fix Playground admin signup: proxy OpenWebUI /workspace+/auth and route /api/v1 via dashboard#884

Merged
Xunzhuo merged 1 commit into
vllm-project:mainfrom
liavweiss:fix/openwebui-api
Dec 23, 2025
Merged

feat: Fix Playground admin signup: proxy OpenWebUI /workspace+/auth and route /api/v1 via dashboard#884
Xunzhuo merged 1 commit into
vllm-project:mainfrom
liavweiss:fix/openwebui-api

Conversation

@liavweiss
Copy link
Copy Markdown
Collaborator

Context

This PR is follow-up work for Issue #849 and builds on the previously-merged fix in PR #850.

After pulling #850, testing showed a remaining issue: clicking Create Admin Account consistently fails with a 500 on the /signup API.

Root cause

OpenWebUI is not fully base-path aware when embedded. In the Playground flow it navigates and/or calls root paths (e.g. /workspace, /auth) and related OpenWebUI APIs. Without explicit dashboard backend routing for these root routes + OpenWebUI API endpoints, requests can fall through to the dashboard SPA or a different upstream, resulting in broken signup/admin creation.

Fix

  • Frontend: Load OpenWebUI in the Playground iframe via a real OpenWebUI root route (/workspace) rather than a base-path mount.
  • Dashboard backend:
    • Proxy OpenWebUI root UI routes needed by the embed (/workspace, /auth) to OpenWebUI.
    • Route OpenWebUI API requests by path (/api/v1/* and /api/config) to OpenWebUI even when the Referer is a root route like /workspace.
    • Preserve Authorization headers when proxying OpenWebUI (forward auth) so authenticated/signup flows work correctly.

Testing

  • http://localhost:8700/playground loads OpenWebUI correctly.
  • Create Admin Account no longer returns 500 and proceeds as expected.

Files changed

  • semantic-router/dashboard/frontend/src/pages/PlaygroundPage.tsx
  • semantic-router/dashboard/backend/router/router.go

…orkspace+/auth, and route OpenWebUI /api/v1

Signed-off-by: Liav Weiss <lweiss@lweiss-thinkpadx1carbongen11.raanaii.csb>
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 22, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 47dadc3
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/69492f5359132c00083205f1
😎 Deploy Preview https://deploy-preview-884--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 dashboard

Owners: @JaredforReal, @Xunzhuo
Files changed:

  • dashboard/backend/router/router.go
  • dashboard/frontend/src/pages/PlaygroundPage.tsx

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

Copy link
Copy Markdown
Member

@Xunzhuo Xunzhuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@Xunzhuo Xunzhuo merged commit 55e78c8 into vllm-project:main Dec 23, 2025
24 checks passed
@liavweiss liavweiss deleted the fix/openwebui-api branch December 24, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants