fix(terraform): remove resource reference in dependent policies (#5684)

* stage * remove user issue * fix unit tests * Fix Unit Tests * stage * update * update comment and code * fix data_source_test_go
Checkmarx · Aug 25, 2022 · 033fb20 · 033fb20
1 parent cbb23fb
commit 033fb20
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 27,856 deletions.
diff --git a/pkg/parser/terraform/data_source.go b/pkg/parser/terraform/data_source.go
@@ -234,10 +234,16 @@ func parseDataSourceBody(body *hclsyntax.Body) string {
 		Variables: inputVariableMap,
 		Functions: functions.TerraformFuncs,
 	})
-	if decodeErrs != nil {
-		log.Debug().Msg("Error trying to eval data source block.")
-		return ""
+
+	// check decode errors
+	for _, decErr := range decodeErrs {
+		if decErr.Summary != "Unknown variable" {
+			log.Debug().Msg("Error trying to eval data source block.")
+			return ""
+		}
+		log.Debug().Msg("Dismissed Error when decoding policy: Found unknown variable")
 	}
+
 	dataSourceJSON := decodeDataSourcePolicy(target)
 	convertedDataSource := convertedPolicy{
 		ID:      dataSourceJSON.ID,

diff --git a/pkg/parser/terraform/variables.go b/pkg/parser/terraform/variables.go
@@ -33,6 +33,7 @@ func setInputVariablesDefaultValues(filename string) (converter.VariableMap, err
 			},
 		},
 	})
+
 	defaultValuesMap := make(converter.VariableMap)
 	for _, block := range content.Blocks {
 		if len(block.Labels) == 0 || block.Labels[0] == "" {
-Original file line number
+Diff line change
@@ Expand Up @@
     			},
     		},
     	})
     	defaultValuesMap := make(converter.VariableMap)
     	for _, block := range content.Blocks {
     		if len(block.Labels) == 0 || block.Labels[0] == "" {
@@ Expand Down @@