pass repo ID to runconfig

bridgecrewio · Jul 9, 2024 · f5af628 · f5af628
1 parent 82c8d8e
commit f5af628
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 3 deletions.
diff --git a/checkov/common/bridgecrew/platform_integration.py b/checkov/common/bridgecrew/platform_integration.py
@@ -989,7 +989,7 @@ def get_platform_run_config(self) -> None:
             self.get_public_run_config()
 
     def _get_run_config_query_params(self) -> str:
-        return f'module={"bc" if self.is_bc_token(self.bc_api_key) else "pc"}&enforcementv2=true'
+        return f'module={"bc" if self.is_bc_token(self.bc_api_key) else "pc"}&enforcementv2=true&repoId={urllib.parse.quote(self.repo_id)}'
 
     def get_run_config_url(self) -> str:
         return f'{self.platform_run_config_url}?{self._get_run_config_query_params()}'

diff --git a/tests/common/integration_features/test_repo_config_integration.py b/tests/common/integration_features/test_repo_config_integration.py
@@ -792,6 +792,27 @@ def test_skip_paths_non_matching(self):
         repo_config_integration._set_exclusion_paths(vcs_config)
         self.assertEqual(repo_config_integration.skip_paths, set())
 
+    def test_skip_paths_no_repos(self):
+        vcs_config = {
+            "scannedFiles": {
+                "sections": [
+                    {
+                        "repos": [],
+                        "rule": {
+                            "excludePaths": []
+                        },
+                        "isDefault": True
+                    }
+                ]
+            }
+        }
+
+        instance = BcPlatformIntegration()
+        instance.repo_id = 'org/repo'
+        repo_config_integration = RepoConfigIntegration(instance)
+        repo_config_integration._set_exclusion_paths(vcs_config)
+        self.assertEqual(repo_config_integration.skip_paths, set())
+
     def test_skip_paths_multiple_one_match(self):
         vcs_config = {
             "scannedFiles": {

diff --git a/tests/common/test_platform_integration.py b/tests/common/test_platform_integration.py
@@ -139,10 +139,13 @@ def test_should_upload(self):
 
     def test_run_config_url(self):
         instance = BcPlatformIntegration()
+        instance.repo_id = 'owner/repo'
         instance.bc_api_key = '00000000-0000-0000-0000-000000000000'
-        self.assertTrue(instance.get_run_config_url().endswith('/runConfiguration?module=bc&enforcementv2=true'))
+        self.assertTrue(instance.get_run_config_url().endswith('/runConfiguration?module=bc&enforcementv2=true&repoId=owner/repo'))
         instance.bc_api_key = '00000000-0000-0000-0000-000000000000::1234=='
-        self.assertTrue(instance.get_run_config_url().endswith('/runConfiguration?module=pc&enforcementv2=true'))
+        self.assertTrue(instance.get_run_config_url().endswith('/runConfiguration?module=pc&enforcementv2=true&repoId=owner/repo'))
+        instance.repo_id = 'encode/më'
+        self.assertTrue(instance.get_run_config_url().endswith('/runConfiguration?module=pc&enforcementv2=true&repoId=encode/m%C3%AB'))
 
     def test_is_valid_policy_filter(self):
         instance = BcPlatformIntegration()