gortium/openconcho
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(web): drop same-origin sentinel from runtime config

Browse Source
This commit is contained in:
Offending Commit
2026-06-02 11:53:42 -05:00
parent 9893230cde
commit b29fa240a6
2 changed files with 11 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
packages/web/src/lib/runtimeConfig.ts
View File

@@ -1,24 +1,17 @@
const GLOBAL_KEY = "__OPENCONCHO_DEFAULT_HONCHO_URL__";
const SAME_ORIGIN = "same-origin";
/**
* Runtime-injected default Honcho base URL for container deployments.
*
* The Docker image writes `/config.js` from the `OPENCONCHO_DEFAULT_HONCHO_URL`
* env at container start, so one prebuilt image can target any backend without
* a rebuild (Vite envs are baked at build time and can't do this).
* The Docker image writes `/config.js` from `OPENCONCHO_DEFAULT_HONCHO_URL` at
* container start, so one prebuilt image can target any backend without a rebuild.
* The web build proxies this URL via the same-origin `/api` reverse proxy (no CORS).
*
* - `"same-origin"` → the page's own origin (pairs with the nginx `/v3` reverse
* proxy, so the browser makes same-origin requests and CORS never applies)
* - an absolute URL → that URL
* - empty / unset → `null` (no default; the user configures in Settings)
* - an absolute URL → that URL (seeds the first instance)
* - empty / unset → null (no default; the user configures in Settings)
*/
export function runtimeDefaultBaseUrl(): string | null {
const raw = (globalThis as Record<string, unknown>)[GLOBAL_KEY];
if (typeof raw !== "string" || raw.trim() === "") return null;
const value = raw.trim();
if (value === SAME_ORIGIN) {
return typeof location !== "undefined" ? location.origin : null;
}
return value;
return raw.trim();
}

18
packages/web/src/test/runtime-config.test.ts
View File

@@ -8,22 +8,14 @@ afterEach(() => {
});
describe("runtimeDefaultBaseUrl", () => {
it("returns null when the global is unset", () => {
expect(runtimeDefaultBaseUrl()).toBeNull();
});
it("returns null when the global is blank", () => {
(globalThis as Record<string, unknown>)[KEY] = " ";
expect(runtimeDefaultBaseUrl()).toBeNull();
});
it("returns an absolute URL verbatim", () => {
it("returns an injected absolute URL verbatim", () => {
(globalThis as Record<string, unknown>)[KEY] = "https://honcho.example.net";
expect(runtimeDefaultBaseUrl()).toBe("https://honcho.example.net");
});
it("resolves 'same-origin' to the page origin", () => {
(globalThis as Record<string, unknown>)[KEY] = "same-origin";
expect(runtimeDefaultBaseUrl()).toBe(location.origin);
it("returns null when unset or empty", () => {
expect(runtimeDefaultBaseUrl()).toBeNull();
(globalThis as Record<string, unknown>)[KEY] = " ";
expect(runtimeDefaultBaseUrl()).toBeNull();
});
});