Skip to content

Flashpoint Release 2.2.1#44447

Open
crestdatasystems wants to merge 1 commit into
demisto:contrib/crestdatasystems_Flashpoint-Release-221from
crestdatasystems:Flashpoint-Release-221
Open

Flashpoint Release 2.2.1#44447
crestdatasystems wants to merge 1 commit into
demisto:contrib/crestdatasystems_Flashpoint-Release-221from
crestdatasystems:Flashpoint-Release-221

Conversation

@crestdatasystems
Copy link
Copy Markdown
Contributor

@crestdatasystems crestdatasystems commented May 28, 2026

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)

Description

Updated Flashpoint Ignite integration

  • Added support for the Enable Exact Match for IOC Enrichment parameter. When enabled, the enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can still be overridden for individual commands using the exact_match argument.

Must have

  • Tests
  • Documentation

relates: https://jira-dc.paloaltonetworks.com/browse/CIAC-16925

@content-bot content-bot added Contribution Form Filled Whether contribution form filled or not. Partner Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack labels May 28, 2026
@content-bot content-bot changed the base branch from master to contrib/crestdatasystems_Flashpoint-Release-221 May 28, 2026 10:15
@content-bot content-bot requested a review from kamalq97 May 28, 2026 10:16
@content-bot
Copy link
Copy Markdown
Contributor

Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @kamalq97 will know the proposed changes are ready to be reviewed.
For your convenience, here is a link to the contributions SLAs document.

@content-bot
Copy link
Copy Markdown
Contributor

Hi @crestdatasystems, thanks for contributing to the XSOAR marketplace. To receive credit for your generous contribution please follow this link.

@content-bot
Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

Hi @kamalq97, you can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@kamalq97 kamalq97 assigned TheL0L and unassigned kamalq97 May 31, 2026
@TheL0L TheL0L requested review from TheL0L and removed request for kamalq97 May 31, 2026 11:28
@TheL0L TheL0L added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label May 31, 2026
@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label May 31, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

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

Hi! Thanks for your contribution to the Flashpoint pack.

I've reviewed the PR and found a few items to address, most notably a critical bug where enabling the exact match parameter causes the email and filename commands to crash due to an argument mismatch. There are also some minor structural updates needed in the YAML and metadata files, along with a few documentation and release note styling tweaks to align with our guidelines.

Let me know if you have any questions!

Additionally, please address the following file-level notes:

  • Packs/Flashpoint/Integrations/Ignite/Ignite.yml: The first key defined at the root level of the .yml file must always be name or display. Currently, the file starts with sectionorder. Please move name or display to the top of the file.
  • Packs/Flashpoint/pack_metadata.json: The keywords list is empty. According to the pack metadata guidelines, the vendor's name (Flashpoint) must appear in the keywords list. Please add it.

@TheL0L please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

type: 8
required: false
section: Collect
- additionalinfo: When enabled, enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can be overridden for individual commands using the exact_match argument.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The description for a boolean parameter must start with 'Whether...'.

Suggested change
- additionalinfo: When enabled, enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can be overridden for individual commands using the exact_match argument.
additionalinfo: Whether to use exact matching for indicator values by default in enrichment commands (ip, domain, file, and url). This behavior can be overridden for individual commands using the exact_match argument.

Additionally, please note a critical bug in the Python implementation: enabling this parameter will cause the email and filename commands to crash. The REPUTATION_COMMAND_TO_FUNCTION loop appends exact_match to the arguments if it is True, but email_lookup_command and filename_lookup_command only accept 2 arguments. Please fix this in the Python code.

Comment thread Packs/Flashpoint/Integrations/Ignite/README.md
Comment thread Packs/Flashpoint/Integrations/Ignite/README.md
Comment thread Packs/Flashpoint/Integrations/Ignite/README.md
Comment thread Packs/Flashpoint/Integrations/Ignite/README.md

##### Flashpoint Ignite

- Added support for the **Enable Exact Match for IOC Enrichment** parameter. When enabled, the enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can still be overridden for individual commands using the exact_match argument.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please update the entity styling to match the documentation guidelines:

  • Parameters and arguments should be in *italics*
  • Command names should be in ***bold-italics***
Suggested change
- Added support for the **Enable Exact Match for IOC Enrichment** parameter. When enabled, the enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can still be overridden for individual commands using the exact_match argument.
- Added support for the *Enable Exact Match for IOC Enrichment* parameter. When enabled, the enrichment commands (***ip***, ***domain***, ***file***, and ***url***) use exact matching for indicator values by default. This behavior can still be overridden for individual commands using the *exact_match* argument.

##### Flashpoint Ignite

- Added support for the **Enable Exact Match for IOC Enrichment** parameter. When enabled, the enrichment commands (ip, domain, file, and url) use exact matching for indicator values by default. This behavior can still be overridden for individual commands using the exact_match argument.
- Updated the Docker image to: *demisto/python3:3.12.13.9059085*.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please update the Docker image styling. Literal values should be in **bold**.

Suggested change
- Updated the Docker image to: *demisto/python3:3.12.13.9059085*.
- Updated the Docker image to: **demisto/python3:3.12.13.9059085**.

@TheL0L TheL0L added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label May 31, 2026
Copy link
Copy Markdown
Contributor

@TheL0L TheL0L left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, @crestdatasystems!

I've finished reviewing the PR and triggered the internal pipeline to run validations and tests on the changes. Please address the comments left by @marketplace-ai-reviewer, as well as the inline comment I left regarding a regression in !email and !filename when exact_match is enabled.

for indicator in indicator_list:
arguments = (client, indicator)
if exact_match:
arguments += (exact_match,) # type: ignore
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The YAML only declares exact_match on domain, ip, url, and file - not on email or filename. However, this loop appends it unconditionally, so when ioc_enrichment_exact_match is enabled, !email and !filename crash with TypeError: ... takes 2 positional arguments but 3 were given.

Could you restrict the exact_match argument to only the commands that actually support it?

Please also add parametrized exact_match tests for email and filename in Ignite_test.py to make sure these commands aren't broken by the config-level exact_match setting going forward.

I would really appreciate it if you could take care of this - thanks!

@TheL0L TheL0L added pending-contributor The PR is pending the response of its creator and removed ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack Partner Partner-Approved pending-contributor The PR is pending the response of its creator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants