Skip to content

Fix/XSUP-69668/ProofpointServerProtection query multiple folders sequentially to avoid engine timeout#44456

Draft
MosheEichler wants to merge 1 commit into
masterfrom
fix/XSUP-69668-proofpoint-multi-folder-sequential-query
Draft

Fix/XSUP-69668/ProofpointServerProtection query multiple folders sequentially to avoid engine timeout#44456
MosheEichler wants to merge 1 commit into
masterfrom
fix/XSUP-69668-proofpoint-multi-folder-sequential-query

Conversation

@MosheEichler
Copy link
Copy Markdown
Contributor

Status

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

Related Issues

https://jira-dc.paloaltonetworks.com/browse/XSUP-69668

Description

The integration now handles multiple folder names internally and sequentially within a single command execution, eliminating the engine-level parallel fan-out entirely.

Must have

  • Tests
  • Documentation

…to avoid engine timeout (XSUP-69668)

When folder_name contains multiple folders (comma-separated or JSON array),
the integration now iterates them sequentially in a single command execution
and aggregates results, instead of relying on the XSOAR engine's parallel
fan-out which hits the hard 5-minute engine response-tracking timeout.
@MosheEichler MosheEichler self-assigned this May 28, 2026
@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

@content-bot
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/ProofpointServerProtection/Integrations/ProofpointProtectionServerV2
   ProofpointProtectionServerV2.py1899151%14, 138, 142–144, 148–150, 154, 167, 170, 201, 210, 225, 322, 335, 347–352, 364, 366–367, 371–379, 381, 383, 394, 396, 400–407, 419, 421–422, 426–433, 435, 437, 442, 444–445, 449–454, 461, 467–486, 488–489
TOTAL1899151% 

Tests Skipped Failures Errors Time
13 0 💤 0 ❌ 0 🔥 2.376s ⏱️

@content-bot
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 1ed5fa298cd7e5ef

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

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