Execute JSON.parse() twice to parse escaped JSON string in `Kubernete…

…sService#getContentScope` (v5) (#2680)

Co-authored-by: Thomas Dax <[email protected]>

.changeset/kind-sloths-relate.md

@@ -0,0 +1,5 @@
+---
+"@comet/cms-api": patch
+---
+
+Fix parsing of `contentScopeAnnotation` in `KubernetesService#getContentScope`

packages/api/cms-api/src/kubernetes/kubernetes.service.ts

@@ -151,6 +151,19 @@ export class KubernetesService {
 
     getContentScope(resource: V1Job | V1CronJob): ContentScope {
         const contentScopeAnnotation = resource.metadata?.annotations?.[CONTENT_SCOPE_ANNOTATION];
-        return contentScopeAnnotation ? JSON.parse(contentScopeAnnotation) : {};
+
+        if (contentScopeAnnotation) {
+            let json = JSON.parse(contentScopeAnnotation);
+
+            // the contentScopeAnnotation is an escaped json string (e.g. "{ \"domain\": \"main\", \"language\": \"en\" }")
+            // therefore JSON.parse() must be executed twice (https://stackoverflow.com/a/25721227)
+            if (typeof json !== "object") {
+                json = JSON.parse(json);
+            }
+
+            return json;
+        }
+
+        return {};
     }
 }