feat(deploy): INDEXERUI_BASE_PATH build-arg + bump indexer-ui submodule#310
feat(deploy): INDEXERUI_BASE_PATH build-arg + bump indexer-ui submodule#310Ahmath-Gadji merged 2 commits intodevfrom
Conversation
Lets operators mount indexer-ui under a subpath (e.g. `/indexerui/`) on the same vhost as the OpenRag backend, eliminating the cross-origin cookie problem when front and back were on different hosts. Changes: - docker-compose.yaml: forward INDEXERUI_BASE_PATH env as BASE_PATH build-arg to the indexer-ui Dockerfile. - .env.example: document INDEXERUI_BASE_PATH (commented, empty = root, rebuild required after change). - extern/indexer-ui: bump pointer to the merged main of openrag-admin-ui (c967017 — OIDC support). Pairs with linagora/openrag-admin-ui#19 (base-path support in the SvelteKit app). Merging that PR + a subsequent submodule pointer bump are required to actually deploy under a subpath. Without the PR merged, setting INDEXERUI_BASE_PATH has no effect (the build-arg is passed but the Dockerfile it reaches doesn't honor it yet).
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 24 minutes and 44 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThese changes enable serving indexer-ui under an optional subpath by updating environment variable documentation with new Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Add a §3 "Mount the Indexer UI under a Subpath" section to setup_indexerui.md covering the single-vhost use case (same-origin cookie for AUTH_MODE=oidc), the build-arg semantics and rebuild requirement, and an example nginx config for the /indexerui/ subpath.
Summary
Enables deploying indexer-ui under a subpath (e.g.
/indexerui/) on the same vhost as the OpenRag backend. Needed by environments behind a single TLS-terminating reverse proxy where we want a single origin to avoid cross-origin cookie handling (and the SameSite pain that goes with it).Changes
docker-compose.yaml(serviceindexer-ui): forwardINDEXERUI_BASE_PATHenv to the container build as theBASE_PATHDocker build-arg..env.example: documentINDEXERUI_BASE_PATH(commented out, empty = root-level as today, rebuild required after change).extern/indexer-ui: bump submodule pointer toc967017(the merged OIDC support onopenrag-admin-ui@main).Dependency
Pairs with linagora/openrag-admin-ui#19 (
feat(base-path): subpath mounting via BASE_PATH env). Without that PR merged, settingINDEXERUI_BASE_PATHin.envhas no effect — the build-arg is passed through compose but the Dockerfile it reaches doesn't read it yet. Merge #19, then bump the submodule pointer again in a follow-up to actually enable the feature.Usage once both PRs land
Reverse proxy (nginx):
Test plan
docker compose config indexer-ui— build.args.BASE_PATH visibleINDEXERUI_BASE_PATH(default) → image builds with empty base, app still at rootINDEXERUI_BASE_PATH=/indexerui→ image builds withbase=/indexerui, app reachable at<host>/indexerui/through the proxy rule aboveSummary by CodeRabbit
New Features
Documentation