Skip to content

Configurable Source AVIT ID Key with Migration Support#70

Open
cx-apoorva-singh wants to merge 4 commits intopre_release_2.0.1from
source_avit_id_multiple_option
Open

Configurable Source AVIT ID Key with Migration Support#70
cx-apoorva-singh wants to merge 4 commits intopre_release_2.0.1from
source_avit_id_multiple_option

Conversation

@cx-apoorva-singh
Copy link
Copy Markdown
Contributor

Configurable Source AVIT ID Key for SAST Findings with Migration Support & Aggregation Enhancements

Summary

This PR introduces a configurable Source AVIT ID Key mechanism for SAST findings, enabling customers to control how source_avit_id is generated. It also provides a migration path from legacy key formats to the new format, along with enhancements to improve visibility and aggregation of related findings.

Key Features & Enhancements

  1. Configurable Source AVIT ID Key
    Added a new Configure Source AVIT ID Key option.
    Allows customers to define how source_avit_id is computed for SAST findings.
  2. Migration Support
    Introduced:
    Select Old Source AVIT ID Key
    Migrate existing AVITs to New Source AVIT ID (toggle)
    When enabled:
    Existing AVITs using the old key format are re-keyed to the new format during the next integration run.
    Migration uses the selected old key format for accurate lookup.
  3. Aggregation of Findings
    When multiple SAST findings share the same source_avit_id:
    Their links are aggregated in the Source Vulnerability Summary field.
    A maximum of 30 links are appended (existingCounter <= 30).
    The Dependency Type field reflects the count of such findings.
  4. Idempotent Scan Behavior
    Re-triggering the same scan:
    Does not duplicate links in the Source Vulnerability Summary.
    Does not incorrectly increment the Dependency Type count.
  5. Validation Enhancements
    Save & Test validation ensures:
    Configure Source AVIT ID Key cannot be empty.
    When migration is enabled:
    Select Old Source AVIT ID Key cannot be empty.
    Old and new key selections cannot be the same.
  6. Migration Output Update
    During migration:
    Full URL display text in source_vulnerability_summary is replaced with resultHash.
  7. Scope Limitation
    Changes apply only to SAST findings.
    Other scan types remain unaffected:
    SCA
    KICS
    Containers
    Secret Detection
    ScoreCard

…9e510026f72021153af1b.js

Removed the aggregate lookup when simiId + resultHash is selected. If simiId + resultHash is selected and apply_migration is true, then the lookup will be only performed on existing data.
removal of dateGenerate function from closureIntegration and also if apply_migration is true creating parameter for latest scan even if old scanId is not found to close those findings which was not migrated because of duplicate source_avit_id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants