mirror of
https://github.com/tiennm99/ccs.git
synced 2026-07-16 08:17:11 +00:00
fix: reject unsupported image analysis backends (#1387)
* fix: reject unsupported image analysis backends * style: apply prettier formatting
This commit is contained in:
@@ -404,6 +404,9 @@ router.put('/', async (req: Request, res: Response): Promise<void> => {
|
||||
if (!normalizedBackend || normalizedModel.length === 0) {
|
||||
return acc;
|
||||
}
|
||||
if (!knownBackends.has(normalizedBackend)) {
|
||||
throw new Error(`Unsupported provider backend "${backendId}".`);
|
||||
}
|
||||
acc[normalizedBackend] = normalizedModel;
|
||||
return acc;
|
||||
},
|
||||
@@ -470,6 +473,10 @@ router.put('/', async (req: Request, res: Response): Promise<void> => {
|
||||
|
||||
res.json(await buildDashboardPayload());
|
||||
} catch (error) {
|
||||
if (error instanceof Error && error.message.startsWith('Unsupported provider backend')) {
|
||||
res.status(400).json({ error: error.message });
|
||||
return;
|
||||
}
|
||||
res.status(500).json({ error: (error as Error).message });
|
||||
}
|
||||
});
|
||||
|
||||
@@ -340,4 +340,31 @@ describe('image-analysis routes', () => {
|
||||
error: 'Profile mapping for "codexProfile" references an unknown backend.',
|
||||
});
|
||||
});
|
||||
|
||||
it('rejects unsupported provider backends and preserves existing configuration', async () => {
|
||||
const putResponse = await fetch(`${baseUrl}/api/image-analysis`, {
|
||||
method: 'PUT',
|
||||
headers: { 'Content-Type': 'application/json' },
|
||||
body: JSON.stringify({
|
||||
providerModels: {
|
||||
gemini: 'gemini-3-flash-preview',
|
||||
evil: 'vision-model',
|
||||
},
|
||||
fallbackBackend: 'gemini',
|
||||
}),
|
||||
});
|
||||
|
||||
expect(putResponse.status).toBe(400);
|
||||
expect(await putResponse.json()).toEqual({
|
||||
error: 'Unsupported provider backend "evil".',
|
||||
});
|
||||
|
||||
const getResponse = await fetch(`${baseUrl}/api/image-analysis`);
|
||||
expect(getResponse.status).toBe(200);
|
||||
const payload = await getResponse.json();
|
||||
expect(payload.config.providerModels).toMatchObject({
|
||||
gemini: 'gemini-3-flash-preview',
|
||||
ghcp: 'claude-haiku-4.5',
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user