Skip to content

Build search indicator query from arguments. #40648

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

omerKarkKatz
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

@omerKarkKatz omerKarkKatz requested a review from dantavori July 20, 2025 16:42
Copy link

github-actions bot commented Jul 20, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/SearchIndicatorAgentix
   SearchIndicatorAgentix.py392146%22, 40–41, 44–51, 53–55, 59–62, 69–71
TOTAL392146% 

Tests Skipped Failures Errors Time
3 0 💤 0 ❌ 0 🔥 3.026s ⏱️

Copy link
Contributor

@dantavori dantavori left a comment

Choose a reason for hiding this comment

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

approving but see notes

type: python
fromversion: 6.5.0
tests:
- No tests (auto formatted)
Copy link
Contributor

Choose a reason for hiding this comment

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

please add:

marketplaces:
  - platform
supportedModules:
  - agentix


##### New: SearchIndicatorAgentix

- New: Added a new script- SearchIndicatorAgentix that Searches Cortex XSOAR Indicators.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- New: Added a new script- SearchIndicatorAgentix that Searches Cortex XSOAR Indicators.
- New: This script searches indicators by specified fields.

@@ -0,0 +1,38 @@
Search for Indicators and returns the id, indicator_type, value, expiration status, lastSeen timestamp, related investigations and score/verdict.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Search for Indicators and returns the id, indicator_type, value, expiration status, lastSeen timestamp, related investigations and score/verdict.
This script searches indicators by specified fields.

Comment on lines +21 to +25
Searches Cortex XSOAR Indicators.

Search for XSOAR Indicators and returns the id, indicator_type, value, expiration status, lastSeen timestamp, related investigations and score/verdict.

You can add additional fields from the indicators using the add_field_to_context argument.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Searches Cortex XSOAR Indicators.
Search for XSOAR Indicators and returns the id, indicator_type, value, expiration status, lastSeen timestamp, related investigations and score/verdict.
You can add additional fields from the indicators using the add_field_to_context argument.
This script searches indicators by specified fields.

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.19.90.

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.

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

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

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.

3 participants