Skip to content

Commit e2ae3d2

Browse files
localai-botlocalai-bot
authored andcommitted
fix: address Copilot review feedback - deduplicate API keys and remove UI-only field
- Use Set to deduplicate API keys before saving - Delete api_keys_text from payload before sending to backend - Addresses feedback from Copilot review comments Signed-off-by: localai-bot <localai-bot@localai.io>
1 parent 97b3ef0 commit e2ae3d2

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

core/http/react-ui/src/pages/Settings.jsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,14 @@ export default function Settings() {
6060
if (settingsToSave.api_keys_text !== undefined && settingsToSave.api_keys_text !== null) {
6161
const text = settingsToSave.api_keys_text
6262
if (typeof text === 'string' && text.trim() !== '') {
63-
settingsToSave.api_keys = text.split('\n').map(k => k.trim()).filter(k => k !== '')
63+
const keys = text.split('\n').map(k => k.trim()).filter(k => k !== '')
64+
settingsToSave.api_keys = Array.from(new Set(keys))
6465
} else {
6566
settingsToSave.api_keys = []
6667
}
6768
}
69+
// Remove UI-only field before sending to backend
70+
delete settingsToSave.api_keys_text
6871
await settingsApi.save(settingsToSave)
6972
addToast('Settings saved successfully', 'success')
7073
} catch (err) {

0 commit comments

Comments
 (0)