mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-26 09:52:11 +03:00
* chore(release): open v3.8.14 development cycle Version bump 3.8.13 -> 3.8.14 (root + electron + open-sse + openapi + lockfiles). Seed the v3.8.14 changelog with the four post-tag hotfixes that shipped to Docker/Electron in v3.8.13 but missed the immutable npm 3.8.13 (#3336 SSRF / CodeQL #323, #3334/#3335/#3339 Electron packaging). i18n CHANGELOG mirrors get the in-progress placeholder section. * feat: add per-provider custom headers support for OpenAI/Anthropic-compatible nodes (#3338) Integrated into release/v3.8.14 * fix: Kiro Builder ID token import fails with Bad credentials (#3333) Integrated into release/v3.8.14 — adds Builder ID cached-creds + OIDC refresh path for Kiro token import, with regression tests (#3333). * Improve code quality: auto-pr/docstrings-1780792063 (#3337) Integrated into release/v3.8.14 — docstring for context analytics route re-export. * fix(catalog): remove minimaxai/minimax-m3 from NVIDIA NIM tier (404 upstream) (#3329) (#3341) NVIDIA NIM does not host minimaxai/minimax-m3 — every request returns 404 page not found, while sibling minimaxai/minimax-m2.7 on the same provider works. Advertising a model that 404s is a catalog bug; remove it from the nvidia tier (it remains on the tiers that actually serve MiniMax M3). Re-add only once NVIDIA serves it. Co-authored-by: mikmaneggahommie <mikmaneggahommie@users.noreply.github.com> * fix(cli): write OpenCode config to ~/.config on all platforms incl. Windows (#3330) (#3343) resolveOpencodeConfigDir used %APPDATA% on Windows, but OpenCode reads its config from XDG ~/.config/opencode/ on every platform (on Windows: %USERPROFILE%\.config\opencode\, NOT %APPDATA%). So a Windows user who configured OpenCode via the dashboard had the file written where OpenCode never looks — it silently had no effect. Use the XDG path (XDG_CONFIG_HOME || ~/.config) unconditionally. Update the UI note + route JSDoc, and flip the three tests that encoded the old %APPDATA% behavior (t40 per-platform + card-note, cli-runtime-extended getCliConfigPaths). Co-authored-by: abdulkadirozyurt <abdulkadirozyurt@users.noreply.github.com> * fix(proxy): make auto-selection fallback opt-in (#3332) (#3344) selectWorkingProxyFallback (Step 11 of resolveProxyForConnection) listed ALL registry proxies, ignoring assignments and per-connection proxy_enabled, and returned the first working one with level:'autoSelect'. So a single proxy added to the registry silently became a global fallback for every connection's traffic. Gate it behind a new PROXY_AUTO_SELECT_ENABLED feature flag (default off): the fallback now no-ops unless the operator opts in. No registry proxy becomes a silent global default anymore. Co-authored-by: hertznsk <hertznsk@users.noreply.github.com> * fix(sse): treat MiniMax M3 as multimodal so vision isn't stripped (#3328) (#3342) MiniMax M3 via the opencode provider (oc/minimax-m3-free) appeared blind: image inputs didn't reach the model, while the same model in Cline could see them. Verified empirically that MiniMax M3 on the opencode upstream IS multimodal -- a base64 image is described correctly (it returns 403 only for remote image URLs, which it doesn't accept). Root cause: OmniRoute treated MiniMax M3 as a non-vision model in two places, so when compression was active the image was replaced with a text placeholder before dispatch: - compression's modelSupportsVision() heuristic (lite.ts) only matched gpt-4/4o/claude-3/gemini/vision -- minimax was absent -> replaceImageUrls stripped the image. - the opencode minimax-m3-free catalog entry lacked supportsVision, so the combo vision-capability gate could also exclude/mishandle it. Add 'minimax-m3' to the vision heuristic and supportsVision: true to the opencode minimax-m3-free entry. TDD: a failing-then-passing test in compression/lite.test.ts proves replaceImageUrls now keeps images for minimax-m3 ids, plus a registry assertion mirroring the #2822 qwen test. Reported-by: @mikmaneggahommie * docs(i18n): translate 25 core documentation files to Indonesian (#3348) Integrated into release/v3.8.14 — Indonesian i18n docs. * fix(review): resolve /review-reviews battery findings (LEDGER-1..11) on v3.8.14 (#3350) Integrated into release/v3.8.14 — /review-reviews battery hardening (LEDGER-1..11) for #3338 custom-headers + #3333 kiro, plus cycle-test drift fixes (#3329/#3330/#3332). * fix(provider-proxy): honor per-account proxy toggles (#3349) Integrated into release/v3.8.14 — honor per-account proxy toggles + auto-fallback opt-in via PROXY_AUTO_SELECT_ENABLED. * fix(dashboard): remove duplicate Distribute Proxies button on provider page (#3352) * fix(providers): reduce proxy label noise (#3346) Integrated into release/v3.8.14 — reduce proxy label noise + a11y (aria-label/sr-only). * fix(duckduckgo): restore bare Response contract and rebase onto release/v3.8.14 (#3323) Integrated into release/v3.8.14 — browser-backed cookie providers (duckduckgo/claude-web) with restored executor contract + unit tests. * fix(noauth): expose only usable model aliases (#3345) Integrated into release/v3.8.14 — noauth usable-alias filtering + registry alias plumbing (veo-free). * fix(dashboard): stop infinite config-load loop on Hermes Agent detail page (#3353) * fix(electron): tree-kill the server on exit/update to release the omniroute.exe lock (#3347) (#3354) * chore(release): finalize v3.8.14 changelog + clear release-gate drift - CHANGELOG: finalize the v3.8.14 section (date, full New Features/Bug Fixes/ Maintenance coverage of all 16 cycle commits, Contributors hall of 12). - docs: document OMNIROUTE_BROWSER_POOL + WEB_COOKIE_USE_BROWSER (#3323) in .env.example + ENVIRONMENT.md; regenerate the id/llm.txt strict mirror (#3348 had translated it; llm.txt mirrors must match root). - test(proxy-fetch): #3323 made tlsClient.available a computed getter — stub it via Object.defineProperty instead of assignment (5 tests were red on the base). * fix(translator): coerce Gemini functionDeclaration parameters to an OBJECT schema (#3357) (#3360) * fix(gemini): resolve truncation/suppression of false positive textual tool call markers in backticks (#3358) Integrated into release/v3.8.14 — Gemini/Antigravity textual tool-call marker normalization (no false-positive suppression + split-chunk buffering). * docs(changelog): add #3358 Gemini textual tool-call normalization to v3.8.14 * fix(dashboard): surface real analytics error instead of generic placeholder (#3356) (#3361) The Analytics page discarded the server's error body on a non-OK response and rendered a generic "An error occurred", so users (and maintainers) could not see why /api/usage/analytics 500'd after an upgrade. Now the route returns the real reason via buildErrorBody (sanitized, Hard Rule #12) and the page surfaces it via a new readFetchErrorMessage helper that handles both the OpenAI-style and legacy error shapes. Reported-by: @superti4r --------- Co-authored-by: PizzaV <103120356+pizzav-xyz@users.noreply.github.com> Co-authored-by: Someres <168349709+quanturbo@users.noreply.github.com> Co-authored-by: Dong Mengzhe <154944819+Lang-Qiu@users.noreply.github.com> Co-authored-by: mikmaneggahommie <mikmaneggahommie@users.noreply.github.com> Co-authored-by: abdulkadirozyurt <abdulkadirozyurt@users.noreply.github.com> Co-authored-by: hertznsk <hertznsk@users.noreply.github.com> Co-authored-by: Krisna Santosa <54174372+KrisnaSantosa15@users.noreply.github.com> Co-authored-by: Randi <55005611+rdself@users.noreply.github.com> Co-authored-by: Wilson <pedbookmed@gmail.com> Co-authored-by: Paijo <14921983+oyi77@users.noreply.github.com> Co-authored-by: Ardem2025 <ardemb22@gmail.com>
607 lines
19 KiB
TypeScript
607 lines
19 KiB
TypeScript
import test from "node:test";
|
|
import assert from "node:assert/strict";
|
|
import fs from "node:fs";
|
|
import os from "node:os";
|
|
import path from "node:path";
|
|
|
|
const TEST_DATA_DIR = fs.mkdtempSync(path.join(os.tmpdir(), "omniroute-custom-headers-"));
|
|
process.env.DATA_DIR = TEST_DATA_DIR;
|
|
|
|
const core = await import("../../src/lib/db/core.ts");
|
|
const providersDb = await import("../../src/lib/db/providers.ts");
|
|
const providerNodesRoute = await import("../../src/app/api/provider-nodes/route.ts");
|
|
const providerNodesIdRoute = await import("../../src/app/api/provider-nodes/[id]/route.ts");
|
|
const { OPENAI_COMPATIBLE_PREFIX } = await import("../../src/shared/constants/providers.ts");
|
|
const { createProviderNodeSchema, updateProviderNodeSchema } =
|
|
await import("../../src/shared/validation/schemas.ts");
|
|
const { DefaultExecutor } = await import("../../open-sse/executors/default.ts");
|
|
|
|
async function resetStorage() {
|
|
core.resetDbInstance();
|
|
fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
|
|
fs.mkdirSync(TEST_DATA_DIR, { recursive: true });
|
|
}
|
|
|
|
function makeRequest(body: Record<string, unknown>) {
|
|
return new Request("http://localhost/api/provider-nodes", {
|
|
method: "POST",
|
|
headers: { "content-type": "application/json" },
|
|
body: JSON.stringify(body),
|
|
});
|
|
}
|
|
|
|
function makeUpdateRequest(id: string, body: Record<string, unknown>) {
|
|
return new Request(`http://localhost/api/provider-nodes/${id}`, {
|
|
method: "PUT",
|
|
headers: { "content-type": "application/json" },
|
|
body: JSON.stringify(body),
|
|
});
|
|
}
|
|
|
|
test.beforeEach(async () => {
|
|
await resetStorage();
|
|
});
|
|
|
|
test.after(async () => {
|
|
await resetStorage();
|
|
fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
|
|
});
|
|
|
|
test("createProviderNodeSchema accepts valid customHeaders as record of strings", () => {
|
|
const validInputs = [
|
|
{ name: "Test", prefix: "test", apiType: "chat", customHeaders: { "X-Custom-1": "value1" } },
|
|
{
|
|
name: "Test",
|
|
prefix: "test",
|
|
apiType: "chat",
|
|
customHeaders: { "X-Header": "value", "X-Another": "value2" },
|
|
},
|
|
{ name: "Test", prefix: "test", apiType: "chat", customHeaders: {} },
|
|
{ name: "Test", prefix: "test", apiType: "chat" },
|
|
];
|
|
|
|
for (const input of validInputs) {
|
|
const result = createProviderNodeSchema.safeParse(input);
|
|
assert.equal(result.success, true, `Should accept: ${JSON.stringify(input)}`);
|
|
}
|
|
});
|
|
|
|
test("createProviderNodeSchema rejects customHeaders with non-string values", () => {
|
|
const invalidInputs = [
|
|
{ customHeaders: { "X-Custom": 123 } },
|
|
{ customHeaders: { "X-Custom": null } },
|
|
{ customHeaders: { "X-Custom": true } },
|
|
{ customHeaders: { "X-Custom": ["array"] } },
|
|
{ customHeaders: { "X-Custom": { nested: "object" } } },
|
|
];
|
|
|
|
for (const input of invalidInputs) {
|
|
const result = createProviderNodeSchema.safeParse(input);
|
|
assert.equal(result.success, false, `Should reject: ${JSON.stringify(input)}`);
|
|
}
|
|
});
|
|
|
|
test("createProviderNodeSchema rejects forbidden hop-by-hop headers", () => {
|
|
const forbiddenHeaders = [
|
|
"host",
|
|
"connection",
|
|
"content-length",
|
|
"keep-alive",
|
|
"proxy-connection",
|
|
"transfer-encoding",
|
|
"te",
|
|
"trailer",
|
|
"upgrade",
|
|
];
|
|
|
|
for (const header of forbiddenHeaders) {
|
|
const result = createProviderNodeSchema.safeParse({
|
|
customHeaders: { [header]: "value" },
|
|
});
|
|
assert.equal(result.success, false, `Should reject forbidden header: ${header}`);
|
|
}
|
|
|
|
const result = createProviderNodeSchema.safeParse({
|
|
customHeaders: { HOST: "evil", "Content-Length": "999" },
|
|
});
|
|
assert.equal(result.success, false, "Should reject case-insensitive forbidden headers");
|
|
});
|
|
|
|
test("updateProviderNodeSchema accepts valid customHeaders", () => {
|
|
const validInputs = [
|
|
{ customHeaders: { "X-Updated": "new-value" } },
|
|
{ customHeaders: { "X-A": "v1", "X-B": "v2" } },
|
|
{ customHeaders: null },
|
|
{},
|
|
];
|
|
|
|
for (const input of validInputs) {
|
|
const result = updateProviderNodeSchema.safeParse({
|
|
name: "Test",
|
|
prefix: "test",
|
|
baseUrl: "https://test.com",
|
|
...input,
|
|
});
|
|
assert.equal(result.success, true, `Should accept: ${JSON.stringify(input)}`);
|
|
}
|
|
});
|
|
|
|
test("updateProviderNodeSchema rejects forbidden headers", () => {
|
|
const result = updateProviderNodeSchema.safeParse({
|
|
name: "Test",
|
|
prefix: "test",
|
|
baseUrl: "https://test.com",
|
|
customHeaders: { host: "evil.com" },
|
|
});
|
|
assert.equal(result.success, false);
|
|
});
|
|
|
|
test("provider nodes route creates OpenAI-compatible nodes with customHeaders", async () => {
|
|
const response = await providerNodesRoute.POST(
|
|
makeRequest({
|
|
name: "Custom Headers Node",
|
|
prefix: "custom-headers",
|
|
apiType: "chat",
|
|
baseUrl: "https://custom.example.com/v1",
|
|
customHeaders: {
|
|
"X-Custom-Auth": "auth-token-123",
|
|
"X-Request-ID": "req-abc",
|
|
},
|
|
})
|
|
);
|
|
const body = (await response.json()) as any;
|
|
|
|
assert.equal(response.status, 201);
|
|
assert.match(body.node.id, new RegExp(`^${OPENAI_COMPATIBLE_PREFIX}chat-`));
|
|
assert.deepEqual(body.node.customHeaders, {
|
|
"X-Custom-Auth": "auth-token-123",
|
|
"X-Request-ID": "req-abc",
|
|
});
|
|
});
|
|
|
|
test("provider nodes route creates nodes without customHeaders (null)", async () => {
|
|
const response = await providerNodesRoute.POST(
|
|
makeRequest({
|
|
name: "No Custom Headers",
|
|
prefix: "no-custom",
|
|
apiType: "chat",
|
|
baseUrl: "https://nocustom.example.com/v1",
|
|
})
|
|
);
|
|
const body = (await response.json()) as any;
|
|
|
|
assert.equal(response.status, 201);
|
|
assert.equal(body.node.customHeaders, null);
|
|
});
|
|
|
|
test("provider nodes route update modifies customHeaders", async () => {
|
|
const createResponse = await providerNodesRoute.POST(
|
|
makeRequest({
|
|
name: "Original Node",
|
|
prefix: "original",
|
|
apiType: "chat",
|
|
baseUrl: "https://original.example.com/v1",
|
|
customHeaders: { "X-Original": "original-value" },
|
|
})
|
|
);
|
|
const created = (await createResponse.json()) as any;
|
|
const nodeId = created.node.id;
|
|
|
|
const updateBody = {
|
|
name: "Updated Node",
|
|
prefix: "updated",
|
|
apiType: "chat",
|
|
baseUrl: "https://updated.example.com/v1",
|
|
customHeaders: { "X-Updated": "updated-value", "X-New": "new-header" },
|
|
};
|
|
const updateResponse = await providerNodesIdRoute.PUT(makeUpdateRequest(nodeId, updateBody), {
|
|
params: Promise.resolve({ id: nodeId }),
|
|
});
|
|
const updated = (await updateResponse.json()) as any;
|
|
|
|
assert.equal(updateResponse.status, 200);
|
|
assert.deepEqual(updated.node.customHeaders, {
|
|
"X-Updated": "updated-value",
|
|
"X-New": "new-header",
|
|
});
|
|
});
|
|
|
|
test("provider nodes route update can clear customHeaders by passing null", async () => {
|
|
const createResponse = await providerNodesRoute.POST(
|
|
makeRequest({
|
|
name: "Node With Headers",
|
|
prefix: "with-headers",
|
|
apiType: "chat",
|
|
baseUrl: "https://withheaders.example.com/v1",
|
|
customHeaders: { "X-Keep": "keep-value" },
|
|
})
|
|
);
|
|
const created = (await createResponse.json()) as any;
|
|
const nodeId = created.node.id;
|
|
|
|
const clearBody = {
|
|
name: "Node Without Headers",
|
|
prefix: "no-headers",
|
|
apiType: "chat",
|
|
baseUrl: "https://noclear.example.com/v1",
|
|
customHeaders: null,
|
|
};
|
|
const updateResponse = await providerNodesIdRoute.PUT(makeUpdateRequest(nodeId, clearBody), {
|
|
params: Promise.resolve({ id: nodeId }),
|
|
});
|
|
const updated = (await updateResponse.json()) as any;
|
|
|
|
assert.equal(updateResponse.status, 200);
|
|
assert.equal(updated.node.customHeaders, null);
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders applies customHeaders from providerSpecificData", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: {
|
|
"X-Custom-Auth": "custom-auth-value",
|
|
"X-Request-ID": "request-123",
|
|
"X-Custom-Header": "extra-value",
|
|
},
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["X-Custom-Auth"], "custom-auth-value");
|
|
assert.equal(headers["X-Request-ID"], "request-123");
|
|
assert.equal(headers["X-Custom-Header"], "extra-value");
|
|
assert.equal(headers["Content-Type"], "application/json");
|
|
assert.equal(headers["Authorization"], "Bearer test-key");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders does NOT override auth headers with customHeaders", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "real-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: {
|
|
Authorization: "Bearer fake-token",
|
|
"x-api-key": "fake-key",
|
|
"X-Custom": "custom-value",
|
|
},
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers.Authorization, "Bearer real-key");
|
|
assert.equal(headers["x-api-key"], undefined);
|
|
assert.equal(headers["X-Custom"], "custom-value");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders blocks forbidden hop-by-hop headers from customHeaders", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: {
|
|
host: "evil.com",
|
|
"content-length": "999",
|
|
connection: "close",
|
|
"X-Legitimate": "good-header",
|
|
},
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers.host, undefined);
|
|
assert.equal(headers["content-length"], undefined);
|
|
assert.equal(headers.connection, undefined);
|
|
assert.equal(headers["X-Legitimate"], "good-header");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders handles string customHeaders (JSON parsed)", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: JSON.stringify({ "X-From-String": "parsed-value" }),
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["X-From-String"], "parsed-value");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders handles invalid JSON in string customHeaders gracefully", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: "not valid json {",
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["X-From-String"], undefined);
|
|
assert.equal(headers.Authorization, "Bearer test-key");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders handles array customHeaders gracefully", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: ["array", "not", "valid"],
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["X-From-String"], undefined);
|
|
assert.equal(headers.Authorization, "Bearer test-key");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders handles null/undefined/empty customHeaders", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const nullHeaders = executor.buildHeaders(
|
|
{ apiKey: "key", providerSpecificData: { customHeaders: null } },
|
|
true
|
|
) as Record<string, string>;
|
|
const undefinedHeaders = executor.buildHeaders(
|
|
{ apiKey: "key", providerSpecificData: {} },
|
|
true
|
|
) as Record<string, string>;
|
|
const emptyHeaders = executor.buildHeaders(
|
|
{ apiKey: "key", providerSpecificData: { customHeaders: {} } },
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
for (const headers of [nullHeaders, undefinedHeaders, emptyHeaders]) {
|
|
assert.equal(headers["X-Anything"], undefined);
|
|
assert.equal(headers.Authorization, "Bearer key");
|
|
}
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders customHeaders are case-sensitive for header names", () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://proxy.example.com/v1",
|
|
customHeaders: {
|
|
"x-lower": "lower-value",
|
|
"X-Upper": "upper-value",
|
|
"X-Mixed": "mixed-value",
|
|
},
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["x-lower"], "lower-value");
|
|
assert.equal(headers["X-Upper"], "upper-value");
|
|
assert.equal(headers["X-Mixed"], "mixed-value");
|
|
});
|
|
|
|
test("DefaultExecutor.buildHeaders for non-openai-compatible providers ignores customHeaders", () => {
|
|
const executor = new DefaultExecutor("openai");
|
|
|
|
const headers = executor.buildHeaders(
|
|
{
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
customHeaders: {
|
|
"X-Custom": "should-be-ignored",
|
|
},
|
|
},
|
|
},
|
|
true
|
|
) as Record<string, string>;
|
|
|
|
assert.equal(headers["X-Custom"], undefined);
|
|
assert.equal(headers.Authorization, "Bearer test-key");
|
|
});
|
|
|
|
test("DefaultExecutor.execute sends customHeaders in the actual HTTP request", async () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
const originalFetch = globalThis.fetch;
|
|
let capturedHeaders: Record<string, string> = {};
|
|
|
|
globalThis.fetch = async (_url: string | URL | Request, init: RequestInit = {}) => {
|
|
capturedHeaders = (init.headers as Record<string, string>) || {};
|
|
return new Response(JSON.stringify({ ok: true }), {
|
|
status: 200,
|
|
headers: { "Content-Type": "application/json" },
|
|
});
|
|
};
|
|
|
|
try {
|
|
await executor.execute({
|
|
model: "gpt-4.1",
|
|
body: { messages: [{ role: "user", content: "hi" }] },
|
|
stream: false,
|
|
credentials: {
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://test.proxy.com/v1",
|
|
customHeaders: {
|
|
"X-Request-ID": "test-req-123",
|
|
"X-Trace-Id": "trace-abc",
|
|
},
|
|
},
|
|
},
|
|
});
|
|
|
|
assert.equal(capturedHeaders["X-Request-ID"], "test-req-123");
|
|
assert.equal(capturedHeaders["X-Trace-Id"], "trace-abc");
|
|
assert.equal(capturedHeaders["Authorization"], "Bearer test-key");
|
|
assert.equal(capturedHeaders["Content-Type"], "application/json");
|
|
} finally {
|
|
globalThis.fetch = originalFetch;
|
|
}
|
|
});
|
|
|
|
test("DefaultExecutor.execute does NOT send forbidden headers from customHeaders in HTTP request", async () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
const originalFetch = globalThis.fetch;
|
|
let capturedHeaders: Record<string, string> = {};
|
|
|
|
globalThis.fetch = async (_url: string | URL | Request, init: RequestInit = {}) => {
|
|
capturedHeaders = (init.headers as Record<string, string>) || {};
|
|
return new Response(JSON.stringify({ ok: true }), {
|
|
status: 200,
|
|
headers: { "Content-Type": "application/json" },
|
|
});
|
|
};
|
|
|
|
try {
|
|
await executor.execute({
|
|
model: "gpt-4.1",
|
|
body: { messages: [{ role: "user", content: "hi" }] },
|
|
stream: false,
|
|
credentials: {
|
|
apiKey: "test-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://test.proxy.com/v1",
|
|
customHeaders: {
|
|
host: "evil.com",
|
|
"content-length": "9999",
|
|
"X-Legitimate": "good",
|
|
},
|
|
},
|
|
},
|
|
});
|
|
|
|
assert.equal(capturedHeaders.host, undefined);
|
|
assert.equal(capturedHeaders["content-length"], undefined);
|
|
assert.equal(capturedHeaders["X-Legitimate"], "good");
|
|
assert.equal(capturedHeaders.Authorization, "Bearer test-key");
|
|
} finally {
|
|
globalThis.fetch = originalFetch;
|
|
}
|
|
});
|
|
|
|
test("DefaultExecutor.execute does NOT allow customHeaders to override Authorization", async () => {
|
|
const executor = new DefaultExecutor("openai-compatible-test");
|
|
const originalFetch = globalThis.fetch;
|
|
let capturedHeaders: Record<string, string> = {};
|
|
|
|
globalThis.fetch = async (_url: string | URL | Request, init: RequestInit = {}) => {
|
|
capturedHeaders = (init.headers as Record<string, string>) || {};
|
|
return new Response(JSON.stringify({ ok: true }), {
|
|
status: 200,
|
|
headers: { "Content-Type": "application/json" },
|
|
});
|
|
};
|
|
|
|
try {
|
|
await executor.execute({
|
|
model: "gpt-4.1",
|
|
body: { messages: [{ role: "user", content: "hi" }] },
|
|
stream: false,
|
|
credentials: {
|
|
apiKey: "real-key",
|
|
providerSpecificData: {
|
|
baseUrl: "https://test.proxy.com/v1",
|
|
customHeaders: {
|
|
Authorization: "Bearer forged-key",
|
|
},
|
|
},
|
|
},
|
|
});
|
|
|
|
assert.equal(capturedHeaders.Authorization, "Bearer real-key");
|
|
assert.notEqual(capturedHeaders.Authorization, "Bearer forged-key");
|
|
} finally {
|
|
globalThis.fetch = originalFetch;
|
|
}
|
|
});
|
|
|
|
test("db: createProviderNode and getProviderNodeById handle customHeaders as JSON", async () => {
|
|
const node = await providersDb.createProviderNode({
|
|
id: "openai-compatible-chat-custom-headers-db",
|
|
type: "openai-compatible",
|
|
name: "DB Custom Headers Test",
|
|
prefix: "db-custom",
|
|
apiType: "chat",
|
|
baseUrl: "https://db.example.com/v1",
|
|
customHeaders: { "X-DB-Header": "db-value", "X-Another": "another" },
|
|
});
|
|
|
|
assert.deepEqual(node.customHeaders, { "X-DB-Header": "db-value", "X-Another": "another" });
|
|
|
|
const retrieved = await providersDb.getProviderNodeById(
|
|
"openai-compatible-chat-custom-headers-db"
|
|
);
|
|
assert.deepEqual(retrieved.customHeaders, { "X-DB-Header": "db-value", "X-Another": "another" });
|
|
});
|
|
|
|
test("db: updateProviderNode modifies customHeaders", async () => {
|
|
const node = await providersDb.createProviderNode({
|
|
id: "openai-compatible-chat-update-custom",
|
|
type: "openai-compatible",
|
|
name: "Update Custom Test",
|
|
prefix: "update-custom",
|
|
apiType: "chat",
|
|
baseUrl: "https://update.example.com/v1",
|
|
customHeaders: { "X-Initial": "initial-value" },
|
|
});
|
|
|
|
assert.deepEqual(node.customHeaders, { "X-Initial": "initial-value" });
|
|
|
|
const updated = await providersDb.updateProviderNode("openai-compatible-chat-update-custom", {
|
|
customHeaders: { "X-Updated": "updated-value", "X-New-Header": "new" },
|
|
});
|
|
|
|
assert.deepEqual(updated.customHeaders, { "X-Updated": "updated-value", "X-New-Header": "new" });
|
|
|
|
const retrieved = await providersDb.getProviderNodeById("openai-compatible-chat-update-custom");
|
|
assert.deepEqual(retrieved.customHeaders, {
|
|
"X-Updated": "updated-value",
|
|
"X-New-Header": "new",
|
|
});
|
|
});
|
|
|
|
test("db: updateProviderNode can clear customHeaders by passing null", async () => {
|
|
const node = await providersDb.createProviderNode({
|
|
id: "openai-compatible-chat-clear-custom",
|
|
type: "openai-compatible",
|
|
name: "Clear Custom Test",
|
|
prefix: "clear-custom",
|
|
apiType: "chat",
|
|
baseUrl: "https://clear.example.com/v1",
|
|
customHeaders: { "X-ToClear": "clear-me" },
|
|
});
|
|
|
|
assert.deepEqual(node.customHeaders, { "X-ToClear": "clear-me" });
|
|
|
|
const updated = await providersDb.updateProviderNode("openai-compatible-chat-clear-custom", {
|
|
customHeaders: null,
|
|
});
|
|
|
|
assert.equal(updated.customHeaders, null);
|
|
});
|