Repository navigation
Add base path support for the observability dashboard #2378
Replies: 6 comments
|
The dashboard frontend should not hardcode http://localhost:4848. It should derive its API and WebSocket endpoints from same-origin by default, with optional configurable public origin and base-path support for reverse-proxy deployments. |
|
Hi! I’d be interested in taking a pass at this. From what I can tell, the dashboard currently works fine when served from My rough plan would be:
I’m thinking of starting with a draft PR so it’s easier to discuss concrete changes instead of only talking in the abstract. If that direction sounds reasonable, I’ll start there. |
|
@Travor278 There's already 2 PRs that adds the same-origin, i feel it's better you build on one of those PRs and add base-path support instead: |
|
Thanks, that’s super helpful. I’ll take a look at #1111 and #1067 first and see which one is the better base to extend with base-path support, instead of starting from scratch. If it makes sense, I can follow up on one of those PRs with the missing subpath/base-path pieces. |
|
Opened a draft PR for this here: #1159 This follows the same-origin proxy direction from #1111 and focuses on adding base-path support for the observability dashboard, including subpath + reverse-proxy validation. I’ll add the supporting screenshots to the PR next. |
|
#2524 makes the dashboard work under a path prefix such as Written by Opus 5.5 (opus-5.5) through Sprocket. |
Uh oh!
There was an error while loading. Please reload this page.
Please add support for hosting the observability dashboard under a configurable base path, for example:
/agent-browser
Current behavior:
What I need is for the dashboard to work correctly when it is not served from /, including:
It would be helpful if this support included either:
All reactions