apunkt/rowboat
1
0
Fork 0
mirror of https://github.com/rowboatlabs/rowboat.git synced 2026-07-21 21:31:12 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix: gate OpenRouter connection on the authenticated /models/user endpoint

Browse source 
OpenRouter's public /api/v1/models returned the full catalog even with an
invalid key, so Settings showed Connected while the model call failed with
Missing Authentication header. With a key present we now fetch the
account-scoped /models/user with the Bearer token, so a bad key surfaces
as an error instead of a false Connected; no key keeps the public preview.
Other providers untouched. Note: unlike the rest of this PR, this touches
core (models.ts).
This commit is contained in:
Prakhar Pandey 2026-07-13 23:07:01 +05:30
parent 9bf286bfea
commit 1a8367ada4
1 changed files with 14 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

16
apps/x/packages/core/src/models/models.ts
View file

@ -259,8 +259,20 @@ export async function listModelsForProvider(
url = `https://generativelanguage.googleapis.com/v1beta/models?key=${apiKey ?? ""}`;
break;
case "openrouter":
url = "https://openrouter.ai/api/v1/models";
if (apiKey) headers["Authorization"] = `Bearer ${apiKey}`;
// /api/v1/models is a public catalog — it returns the full
// list even with an invalid/absent key, so listing it can't
// tell a bad key from a good one (a false "Connected"). When
// a key is given, hit the account-scoped /models/user behind
// Bearer auth instead: a bad key 401s here and the shared
// throw below surfaces it. Same OpenAI-shaped { data:[{id}] }
// response, so the parse path is unchanged. No key → keep the
// public catalog so an unconfigured provider can still preview.
if (apiKey) {
url = "https://openrouter.ai/api/v1/models/user";
headers["Authorization"] = `Bearer ${apiKey}`;
} else {
url = "https://openrouter.ai/api/v1/models";
}
break;
case "ollama":
url = `${(baseURL ?? "http://localhost:11434").replace(/\/$/, "")}/api/tags`;