From 317f55bb256bd978eb2f10874fcf09ec0f69ac8d Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Fri, 27 Aug 2021 13:28:08 +0200
Subject: [PATCH] ConfigServer: Update the configuration cache on file changes

We were only notifying clients about the change, but didn't actually
update the internal cache in ConfigServer itself.

Thanks to "The Grey One" for pointing this out. :^)
---
 Userland/Services/ConfigServer/ClientConnection.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Userland/Services/ConfigServer/ClientConnection.cpp b/Userland/Services/ConfigServer/ClientConnection.cpp
index 6a2e0194b6..4f5e7b02f4 100644
--- a/Userland/Services/ConfigServer/ClientConnection.cpp
+++ b/Userland/Services/ConfigServer/ClientConnection.cpp
@@ -50,6 +50,8 @@ static Core::ConfigFile& ensure_domain_config(String const& domain)
                 }
             }
         }
+        // FIXME: Refactor this whole thing so that we don't need a cache lookup here.
+        s_cache.get(domain).value()->config = new_config;
     };
     auto cache_entry = make<CachedDomain>(domain, config, watcher_or_error.release_value());
     s_cache.set(domain, move(cache_entry));