fix: use JSON.parse to avoid OOM error

snyk · Dec 5, 2022 · 535225f · 535225f
1 parent 7445bc9
commit 535225f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 9 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -61,7 +61,7 @@
   "dependencies": {
     "@open-policy-agent/opa-wasm": "^1.6.0",
     "@snyk/cli-interface": "2.11.0",
-    "@snyk/cloud-config-parser": "^1.14.3",
+    "@snyk/cloud-config-parser": "^1.14.5",
     "@snyk/code-client": "^4.14.0",
     "@snyk/dep-graph": "^1.27.1",
     "@snyk/docker-registry-v2-client": "^2.7.3",

diff --git a/src/cli/commands/test/iac/local-execution/yaml-parser.ts b/src/cli/commands/test/iac/local-execution/yaml-parser.ts
@@ -2,11 +2,15 @@ import { CustomError } from '../../../../../lib/errors';
 import { getErrorStringCode } from './error-utils';
 import { IaCErrorCodes, IacFileData } from './types';
 import { parseFileContent } from '@snyk/cloud-config-parser';
+import { ParserFileType } from '@snyk/cloud-config-parser/dist/yaml-parser';
 
 export function parseYAMLOrJSONFileData(fileData: IacFileData): any[] {
   try {
     // this function will always be called with the file types recognised by the parser
-    return parseFileContent(fileData.fileContent);
+    return parseFileContent(
+      fileData.fileContent,
+      fileData.fileType as ParserFileType,
+    );
   } catch (e) {
     if (fileData.fileType === 'json') {
       throw new InvalidJsonFileError(fileData.filePath);