feat(terraform): add CKV NCP rules about access control group outbound rule.

.gitignore

@@ -172,4 +172,4 @@ tests/20*
 # vim 
 .vim/
 .vimspector.json
-!tests/terraform/graph/variable_rendering/test_resources/tfvar_module_variables/modules/instance
+!tests/terraform/graph/variable_rendering/test_resources/tfvar_module_variables/modules/instance

checkov/terraform/checks/resource/ncp/AccessControlGroupOutboundRule.py

@@ -0,0 +1,23 @@
+from checkov.terraform.checks.resource.base_resource_check import BaseResourceCheck
+from checkov.common.models.enums import CheckResult, CheckCategories
+
+
+class AccessControlGroupOutboundRule(BaseResourceCheck):
+    def __init__(self):
+        name = "Ensure no security group rules allow outbound traffic to 0.0.0.0/0"
+        id = "CKV_NCP_3"
+        supported_resources = ('ncloud_access_control_group_rule',)
+
+        categories = (CheckCategories.NETWORKING,)
+        super().__init__(name=name, id=id, categories=categories,
+                         supported_resources=supported_resources)
+
+    def scan_resource_conf(self, conf):
+        for outbound in conf.get('outbound', []):
+            ip = outbound.get('ip_block')
+            if ip == ['0.0.0.0/0'] or ip == ['::/0']:
+                return CheckResult.FAILED
+        return CheckResult.PASSED
+
+
+check = AccessControlGroupOutboundRule()

checkov/terraform/checks/resource/ncp/AccessControlGroupRuleDescription.py

@@ -5,7 +5,7 @@
 class AccessControlGroupRuleDescription(BaseResourceCheck):
     def __init__(self):
         name = "Ensure every access control groups rule has a description"
-        id = "CKV_NCP_002"
+        id = "CKV_NCP_2"
         supported_resource = [
             'ncloud_access_control_group',
             'ncloud_access_control_group_rule',

tests/terraform/checks/resource/ncp/example_AccessControlGroupOutboundRule/main.tf

@@ -0,0 +1,53 @@
+resource "ncloud_access_control_group_rule" "pass" {
+  access_control_group_no = ncloud_access_control_group.acg.id
+
+  inbound {
+    protocol    = "TCP"
+    ip_block    = "0.0.0.0/0"
+    port_range  = "22"
+    description = "accept 22 port"
+  }
+
+  outbound {
+    protocol    = "TCP"
+    ip_block    = "10.0.3.0/16" 
+    port_range  = "1-65535"
+    description = "accept 1-65535 port"
+  }
+}
+
+resource "ncloud_access_control_group_rule" "fail" {
+  access_control_group_no = ncloud_access_control_group.acg.id
+
+  inbound {
+    protocol    = "TCP"
+    ip_block    = "10.0.3.0/16"
+    port_range  = "22"
+    description = "accept 22 port"
+  }
+
+  outbound {
+    protocol    = "TCP"
+    ip_block    = "0.0.0.0/0" 
+    port_range  = "1-65535"
+    description = "accept 1-65535 port"
+  }
+}
+
+resource "ncloud_access_control_group_rule" "fail1" {
+  access_control_group_no = ncloud_access_control_group.acg.id
+
+  inbound {
+    protocol    = "TCP"
+    ip_block    = "10.16.0.0/32" 
+    port_range  = "1-65535"
+    description = "accept 1-65535 port"
+  }
+
+  outbound {
+    protocol    = "TCP"
+    ip_block    = "::/0"
+    port_range  = "22"
+    description = "accept 22 port"
+  }
+}

tests/terraform/checks/resource/ncp/test_AccessControlGroupOutboundRule.py

@@ -0,0 +1,41 @@
+import unittest
+from pathlib import Path
+
+from checkov.runner_filter import RunnerFilter
+from checkov.terraform.checks.resource.ncp.AccessControlGroupOutboundRule import check
+from checkov.terraform.runner import Runner
+
+
+class TestAccessControlGroupOutboundRule(unittest.TestCase):
+    def test(self):
+        # given
+        test_files_dir = Path(__file__).parent / "example_AccessControlGroupOutboundRule"
+
+        # when
+        report = Runner().run(root_folder=str(test_files_dir), runner_filter=RunnerFilter(checks=[check.id]))
+
+        # then
+        summary = report.get_summary()
+
+        passing_resources = {
+            "ncloud_access_control_group_rule.pass"
+        }
+        failing_resources = {
+            "ncloud_access_control_group_rule.fail",
+            "ncloud_access_control_group_rule.fail1"
+        }
+
+        passed_check_resources = {c.resource for c in report.passed_checks}
+        failed_check_resources = {c.resource for c in report.failed_checks}
+
+        self.assertEqual(summary["passed"], 1)
+        self.assertEqual(summary["failed"], 2)
+        self.assertEqual(summary["skipped"], 0)
+        self.assertEqual(summary["parsing_errors"], 0)
+
+        self.assertEqual(passing_resources, passed_check_resources)
+        self.assertEqual(failing_resources, failed_check_resources)
+
+
+if __name__ == "__main__":
+    unittest.main()