mirror of
https://github.com/windmill-labs/windmill.git
synced 2026-08-20 08:01:35 +00:00
cab0000f3a
Setting a typed instance setting (e.g. `object_store_cache_config`) to `null` via `PUT /api/settings/instance_config` was silently ignored: `Json<InstanceConfig>` deserialized null to `None`, and `to_settings_map` then dropped it via `skip_serializing_if = "Option::is_none"`, so `diff_global_settings` (Merge mode) never saw the deletion. Add `#[serde(remote = "Self")]` to `GlobalSettings` plus a manual `Deserialize` impl that captures top-level explicit nulls into `extra`, which already round-trips correctly via flatten. The handler stays fully typed. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>