From 8282643db5da4f6cd8b784f46447cfbc386ecc13 Mon Sep 17 00:00:00 2001
From: Anna <anna.yafi@datadoghq.com>
Date: Wed, 10 Apr 2024 17:09:49 +0200
Subject: [PATCH] dc

---
 tracer/src/Datadog.Trace/AppSec/Rcm/ConfigurationStatus.cs | 3 ++-
 tracer/src/Datadog.Trace/AppSec/Security.cs                | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/tracer/src/Datadog.Trace/AppSec/Rcm/ConfigurationStatus.cs b/tracer/src/Datadog.Trace/AppSec/Rcm/ConfigurationStatus.cs
index dee90d181051..0d5bf2e4da65 100644
--- a/tracer/src/Datadog.Trace/AppSec/Rcm/ConfigurationStatus.cs
+++ b/tracer/src/Datadog.Trace/AppSec/Rcm/ConfigurationStatus.cs
@@ -148,7 +148,8 @@ internal Dictionary<string, object> BuildDictionaryForWafAccordingToIncomingUpda
     }
 
     /// <summary>
-    /// Calls each file updater to deserialize all remote config payloads and store them in the proper dictionaries with involved merges
+    /// Calls each file updater to deserialize all remote config payloads and store them in the proper dictionaries with might involve various logical merges
+    /// This method deserializes everything stored in _fileUpdates. ConfigurationStatus will have a *bigger* memory footprint.
     /// </summary>
     public void ApplyStoredFiles()
     {
diff --git a/tracer/src/Datadog.Trace/AppSec/Security.cs b/tracer/src/Datadog.Trace/AppSec/Security.cs
index 37f4f97a3edd..ef1a02effbf9 100644
--- a/tracer/src/Datadog.Trace/AppSec/Security.cs
+++ b/tracer/src/Datadog.Trace/AppSec/Security.cs
@@ -171,6 +171,7 @@ private ApplyDetails[] UpdateFromRcm(Dictionary<string, List<RemoteConfiguration
 
             try
             {
+                // store the last config state, clearing any previous state, without deserializing any payloads yet.
                 var anyChange = _configurationStatus.StoreLastConfigState(configsByProduct, removedConfigs);
                 var securityStateChange = Enabled != _configurationStatus.EnableAsm;