Fix: Panic on data source wiz_cloud_configuration_rules (#33)

* fix panic on data wiz_cloud_configuration_rules if a returned rule does not have a control * disabled github action `tests` on push
AxtonGrams · Oct 27, 2022 · 1f06646 · 1f06646
1 parent f9bdeec
commit 1f06646
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -3,9 +3,6 @@ on:
   pull_request:
     paths-ignore:
       - 'README.md'
-  push:
-    paths-ignore:
-      - 'README.md'
 jobs:
   # ensure the code builds...
   build:

diff --git a/internal/provider/data_source_cloud_configuration_rules.go b/internal/provider/data_source_cloud_configuration_rules.go
@@ -721,7 +721,9 @@ func flattenCloudConfigurationRules(ctx context.Context, nodes *[]*vendor.CloudC
 		ruleMap["builtin"] = *b.Builtin
 		ruleMap["opa_policy"] = b.OPAPolicy
 		ruleMap["function_as_control"] = *b.FunctionAsControl
-		ruleMap["control_id"] = b.Control.ID
+		if b.Control != nil {
+			ruleMap["control_id"] = b.Control.ID
+		}
 		ruleMap["graph_id"] = b.GraphID
 		ruleMap["has_auto_remediation"] = *b.HasAutoRemediation
 		ruleMap["remediation_instructions"] = b.RemediationInstructions