[Synthetics] Normalize monitor before mixing params. (#163176)

Fixes #163042 ## Summary Normalize the monitor object before mixing global and project-wide params.
elastic · Aug 4, 2023 · 339eb28 · 339eb28
1 parent 3efc73c
commit 339eb28
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/...gins/synthetics/server/synthetics_service/synthetics_monitor/synthetics_monitor_client.ts b/...gins/synthetics/server/synthetics_service/synthetics_monitor/synthetics_monitor_client.ts
@@ -361,13 +361,16 @@ export class SyntheticsMonitorClient {
     const heartbeatConfigs: HeartbeatConfig[] = [];
 
     for (const monitor of monitors) {
-      const attributes = monitor.attributes as unknown as MonitorFields;
-      const { str: paramsString } = mixParamsWithGlobalParams(paramsBySpace[spaceId], attributes);
+      const normalizedMonitor = normalizeSecrets(monitor).attributes as MonitorFields;
+      const { str: paramsString } = mixParamsWithGlobalParams(
+        paramsBySpace[spaceId],
+        normalizedMonitor
+      );
 
       heartbeatConfigs.push(
         formatHeartbeatRequest(
           {
-            monitor: normalizeSecrets(monitor).attributes,
+            monitor: normalizedMonitor,
             configId: monitor.id,
           },
           paramsString