bridgecrewio / checkov

Prevent cloud misconfigurations and find vulnerabilities during build-time in infrastructure as code, container images and open source packages with Checkov by Bridgecrew.
https://www.checkov.io/
Apache License 2.0
7.17k stars 1.12k forks source link

fix(arm): Fix arm graph breadcrumbs #6869

Closed omriyoffe-panw closed 5 days ago

omriyoffe-panw commented 5 days ago

User description

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[//]: # "

PR Title

We use the title to create changelog automatically and therefore only allow specific prefixes
- break:    to indicate a breaking change, this supersedes any of the other types
- feat:     to indicate new features or checks
- fix:      to indicate a bugfix or handling of edge cases of existing checks
- docs:     to indicate an update to our documentation
- chore:    to indicate adjustments to workflow files or dependency updates
- platform: to indicate a change needed for the platform
Each prefix should be accompanied by a scope that specifies the targeted framework. If uncertain, use 'general'.
#    
Allowed prefixs:
ansible|argo|arm|azure|bicep|bitbucket|circleci|cloudformation|dockerfile|github|gha|gitlab|helm|kubernetes|kustomize|openapi|sast|sca|secrets|serverless|terraform|general|graph|terraform_plan|terraform_json
#
ex.
feat(terraform): add CKV_AWS_123 to ensure that VPC Endpoint Service is configured for Manual Acceptance

"

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

New/Edited policies (Delete if not relevant)

Description

Include a description of what makes it a violation and any relevant external links.

Fix

How does someone fix the issue in code and/or in runtime?

Checklist:


Generated description

Below is a concise technical summary of the changes proposed in this PR:

Refactor the Runner class in checkov/arm/runner.py to improve file path handling and breadcrumb integration. Introduce clean_file_path in checkov/arm/utils.py to sanitize file paths. Enhance breadcrumb functionality by incorporating GraphRecord for better tracking of resources in the ARM graph.

<table><tr><th>Topic</th><th>Details</th><tr><td><a href=https://baz.co/changes/bridgecrewio/checkov/6869?tool=ast&topic=File+Path+Handling>File Path Handling</a>
    </td><td>Improve file path handling in the <code>Runner</code> class by using <code>clean_file_path</code> to sanitize paths and <code>extract_file_path_from_abs_path</code> to convert absolute paths to relative paths.<details><summary>Modified files (2)</summary><ul><li>checkov/arm/runner.py</li>
  • checkov/arm/utils.py
  • Latest Contributors(2)
    UserCommitDate
    taviassaffeat-azure-add-new-pol...November 17, 2024
    omriyoffe-panwfix-arm-finish-variabl...November 07, 2024
    Breadcrumb Integration Enhance breadcrumb functionality in the Runner class by integrating GraphRecord to track resources more effectively.
    Modified files (1)
    • checkov/arm/runner.py
    Latest Contributors(2)
    UserCommitDate
    omriyoffe-panwfix-arm-finish-variabl...November 07, 2024
    lirshindalmanfeat-arm-Add-arm-defin...October 30, 2024
    This pull request is reviewed by Baz. Join @omriyoffe-panw and the rest of your team on (Baz).