pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-08-05 06:42:12 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: getProviderCredentials missing allowedConnections param (#363 TS error)

Browse Source 
PR #363 added allowedConnections as 3rd arg in chat.ts calls to
getProviderCredentials(), but the function signature in auth.ts
only declared 2 params. Adding the optional 3rd param and applying
the connection filter when provided.
This commit is contained in:
diegosouzapw
2026-03-14 15:38:12 -03:00
parent da14390fe0
commit 4d086542aa
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/sse/services/auth.ts
View File

@@ -101,7 +101,8 @@ export { fisherYatesShuffle, getNextFromDeckSync as getNextFromDeck };
*/
export async function getProviderCredentials(
provider: string,
excludeConnectionId: string | null = null
excludeConnectionId: string | null = null,
allowedConnections: string[] | null = null
) {
// Acquire mutex to prevent race conditions
const currentMutex = selectionMutex;
@@ -114,9 +115,13 @@ export async function getProviderCredentials(
await currentMutex;
const connectionsRaw = await getProviderConnections({ provider, isActive: true });
const connections = (Array.isArray(connectionsRaw) ? connectionsRaw : [])
let connections = (Array.isArray(connectionsRaw) ? connectionsRaw : [])
.map(toProviderConnection)
.filter((conn) => conn.id.length > 0);
// allowedConnections: restrict to specific connection IDs (from API key policy, #363)
if (allowedConnections && allowedConnections.length > 0) {
connections = connections.filter((conn) => allowedConnections.includes(conn.id));
}
log.debug(
"AUTH",
`${provider} | total connections: ${connections.length}, excludeId: ${excludeConnectionId || "none"}`