Skip to content

Add fetch lookback functionality to Palo Alto Enterprise DLP#44223

Merged
kamalq97 merged 3 commits into
masterfrom
ciac-16732-dlp-lookback
May 12, 2026
Merged

Add fetch lookback functionality to Palo Alto Enterprise DLP#44223
kamalq97 merged 3 commits into
masterfrom
ciac-16732-dlp-lookback

Conversation

@kamalq97
Copy link
Copy Markdown
Contributor

@kamalq97 kamalq97 commented May 6, 2026

Related Issues

fixes: CIAC-16732

Description

  • Added support for the Fetch Lookback Window (minutes) parameter, which can be used to configure the number of minutes to look back during each fetch to capture late-indexed incidents.

@kamalq97 kamalq97 self-assigned this May 6, 2026
@kamalq97 kamalq97 added bug python Pull requests that update Python code ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels May 6, 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 6, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@content-bot
Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Palo_Alto_Networks_Enterprise_DLP/Integrations/Palo_Alto_Networks_Enterprise_DLP
   Palo_Alto_Networks_Enterprise_DLP.py3866383%82–84, 100–102, 111, 120–121, 143, 150–151, 177, 190, 355, 401–404, 406–408, 410–411, 439–443, 445, 619, 646–647, 667–668, 671–672, 700–701, 703–705, 707–708, 710, 752, 781–794, 796, 798–799
TOTAL3866383% 

Tests Skipped Failures Errors Time
33 0 💤 0 ❌ 0 🔥 2.868s ⏱️

@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 there! Thanks for contributing to the Palo Alto Networks Enterprise DLP pack. I've reviewed your changes and have a few quick adjustments for you.

Please update the pack metadata to align with XSOAR support standards (author, URL, and keywords) and ensure the integration YAML starts with the name or display key rather than category. In the Python code, consider adding a unit test for the new migration function and updating a stale comment regarding the removed fetch IDs.

Thanks again for your work on this!

Additionally, please address the following file-level notes:

  • Packs/Palo_Alto_Networks_Enterprise_DLP/Integrations/Palo_Alto_Networks_Enterprise_DLP/Palo_Alto_Networks_Enterprise_DLP.yml: The .yml file must start with the name or display key. Currently, it starts with category.
  • Packs/Palo_Alto_Networks_Enterprise_DLP/pack_metadata.json: - The support field is set to xsoar, but the author and url fields do not match the required values for XSOAR-supported packs. Please update them to:
  • author: Cortex XSOAR (or Cortex)
  • url: https://www.paloaltonetworks.com/cortex
  • The keywords list is currently empty. Please add the vendor name (e.g., Palo Alto Networks) to the keywords array.

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

@content-bot

This comment has been minimized.

@kamalq97 kamalq97 marked this pull request as ready for review May 11, 2026 11:11
@kamalq97 kamalq97 requested a review from DeanArbel as a code owner May 11, 2026 11:11
@content-bot

This comment has been minimized.

@content-bot
Copy link
Copy Markdown
Contributor

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? ✅

@content-bot
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: success
Report ID: 4d4c4150459f32f1

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

@kamalq97 kamalq97 merged commit 2a6efcf into master May 12, 2026
22 checks passed
@kamalq97 kamalq97 deleted the ciac-16732-dlp-lookback branch May 12, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug docs-approved python Pull requests that update Python code ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants