Skip to content

Conversation

@almog2296
Copy link
Contributor

Related Issues: fixes CRTX-217480

Description

This PR updates the docs of all enrichment scripts, specifying the supported indicator type and the creation of the indicator in TIM.

@github-actions
Copy link

github-actions bot commented Dec 31, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AggregatedScripts/Scripts/CVEEnrichment
   CVEEnrichment.py140100% 
Packs/AggregatedScripts/Scripts/DomainEnrichment
   DomainEnrichment.py280100% 
Packs/AggregatedScripts/Scripts/FileEnrichment
   FileEnrichment.py220100% 
Packs/AggregatedScripts/Scripts/IPEnrichment
   IPEnrichment.py441272%136–143, 145–148
Packs/AggregatedScripts/Scripts/URLEnrichment
   URLEnrichment.py220100% 
TOTAL1301290% 

Tests Skipped Failures Errors Time
1 0 💤 0 ❌ 0 🔥 2.061s ⏱️

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: GR101, GR110.
The following errors cannot be ignored: GR101, GR110.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: GR110.

Verdict: PR can be force merged from validate perspective? ❌

@almog2296 almog2296 requested a review from Shellyber December 31, 2025 14:25
required: false
defaultValue: 'false'
comment: 'This script gathers for reputation data from multiple integrations and returns a "FileEnrichment" object with consolidated information in the context output. Note: The script runs core-get-hash-analytics-prevalence on SHA256 values only.'
comment: 'This script enriches File indicators using reputation data from multiple integrations and returns a consolidated FileEnrichment object in the context output. It supports only indicators of type File and will create the File indicator in TIM if it does not exist. Note: The script runs core-get-hash-analytics-prevalence on SHA256 values only.'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to specify here which file type we support. In File specifically

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.3.16.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Collaborator

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

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.

5 participants