Skip to content

[-] fix --web-base-path deployments, closes #1328#1329

Merged
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
Syedowais312:fix/web-ui-assest-path
Apr 9, 2026
Merged

[-] fix --web-base-path deployments, closes #1328#1329
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
Syedowais312:fix/web-ui-assest-path

Conversation

@Syedowais312
Copy link
Copy Markdown
Contributor

Description

Fix the embedded web UI so it loads correctly when --web-base-path is set.

Fixes #1328

The frontend build was generating root-relative asset URLs like /assets/ index-*.js. When pgwatch is served under a subpath such as /pgwatch/, the
browser requests /assets/... instead of /pgwatch/assets/..., which causes
the main bundle to 404 and leaves the page blank.

This change sets the Vite build base to a relative path so generated assets
resolve correctly under both root and subpath deployments.

AI & Automation Policy

  • [ ✔️] I am the human author and take full personal responsibility for every change in this PR.
  • No AI or automated generative tool was used in any part of this PR OR I have disclosed all tool(s) below.

AI/automation tools used (leave blank if none):

Checklist

  • [ ✔️] Code compiles and existing tests pass locally.
  • New or updated tests are included where applicable.
  • Documentation is updated where applicable.

@pashagolub pashagolub changed the title Fix web UI assest paths for web-base-path deployments [-] fix --web-base-path deployments, closes #1328 Apr 9, 2026
@pashagolub pashagolub self-assigned this Apr 9, 2026
@pashagolub pashagolub added bug Something isn't working webui Admin UI related issues labels Apr 9, 2026
@pashagolub pashagolub force-pushed the fix/web-ui-assest-path branch from 6d7fd47 to e0cad0f Compare April 9, 2026 15:29
@pashagolub pashagolub merged commit b84b922 into cybertec-postgresql:master Apr 9, 2026
1 check passed
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 24198664038

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage decreased (-0.08%) to 83.408%

Details

  • Coverage decreased (-0.08%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 4 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

4 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
internal/reaper/reaper.go 4 37.04%

Coverage Stats

Coverage Status
Relevant Lines: 5322
Covered Lines: 4439
Line Coverage: 83.41%
Coverage Strength: 0.95 hits per line

💛 - Coveralls

@Syedowais312 Syedowais312 deleted the fix/web-ui-assest-path branch April 9, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working webui Admin UI related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web UI fails to load when --web-base-path is set

3 participants