mirror of
https://github.com/tiennm99/ccs.git
synced 2026-09-03 04:17:54 +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) {
|
if (!normalizedBackend || normalizedModel.length === 0) {
|
||||||
return acc;
|
return acc;
|
||||||
}
|
}
|
||||||
|
if (!knownBackends.has(normalizedBackend)) {
|
||||||
|
throw new Error(`Unsupported provider backend "${backendId}".`);
|
||||||
|
}
|
||||||
acc[normalizedBackend] = normalizedModel;
|
acc[normalizedBackend] = normalizedModel;
|
||||||
return acc;
|
return acc;
|
||||||
},
|
},
|
||||||
@@ -470,6 +473,10 @@ router.put('/', async (req: Request, res: Response): Promise<void> => {
|
|||||||
|
|
||||||
res.json(await buildDashboardPayload());
|
res.json(await buildDashboardPayload());
|
||||||
} catch (error) {
|
} 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 });
|
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.',
|
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