From 4c0feade12e240066172caead967fb204a744db0 Mon Sep 17 00:00:00 2001 From: carlosjarenom Date: Tue, 9 Jun 2026 16:45:01 +0200 Subject: [PATCH] fix: correct PUT /api/v1/provider silently dropping model_type, endpoint_url, encrypted_config (#1627) The _UPDATABLE_FIELDS whitelist used obsolete field names (api_base, extra_config) that no longer exist in the ModelProvider model. When updating a provider via PUT, the actual fields model_type, endpoint_url and encrypted_config were silently ignored. Updated the whitelist to use the correct field names matching the ModelProvider model definition. Closes #1627 --- server/app/domains/model_provider/service/provider_service.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/app/domains/model_provider/service/provider_service.py b/server/app/domains/model_provider/service/provider_service.py index 599d3f72..086bc58b 100644 --- a/server/app/domains/model_provider/service/provider_service.py +++ b/server/app/domains/model_provider/service/provider_service.py @@ -66,7 +66,7 @@ class ProviderService: if not model: return {"success": False, "error_code": "PROVIDER_NOT_FOUND"} # H10: only allow updating safe fields - _UPDATABLE_FIELDS = {"provider_name", "api_key", "api_base", "extra_config", "prefer", "is_vaild"} + _UPDATABLE_FIELDS = {"provider_name", "model_type", "api_key", "endpoint_url", "encrypted_config", "prefer", "is_vaild"} for key, value in data.items(): if key in _UPDATABLE_FIELDS: setattr(model, key, value)