Fix API key exposure in toast notifications, resolves #1477 (#1480)

This commit is contained in:
amodev
2024-05-01 15:01:48 +09:00
committed by GitHub
parent 5f2d6f58f2
commit 62e4fb47b2

View File

@@ -76,8 +76,14 @@ function SettingsModal({ isOpen, onOpenChange }: SettingsProps) {
i18next.changeLanguage(settings.LANGUAGE);
initializeAgent(settings); // reinitialize the agent with the new settings
const sensitiveKeys = ['LLM_API_KEY'];
Object.entries(updatedSettings).forEach(([key, value]) => {
toast.settingsChanged(`${key} set to "${value}"`);
if (!sensitiveKeys.includes(key)) {
toast.settingsChanged(`${key} set to "${value}"`);
} else {
toast.settingsChanged(`${key} has been updated securely.`);
}
});
localStorage.setItem(