Skip to content

ProofpointThreatResponse - Add diagnostic logging and optimize batch fetch loop#42929

Open
DanielTal87 wants to merge 1 commit intomasterfrom
XSUP-61776-ProofpointThreatResponse-timeout
Open

ProofpointThreatResponse - Add diagnostic logging and optimize batch fetch loop#42929
DanielTal87 wants to merge 1 commit intomasterfrom
XSUP-61776-ProofpointThreatResponse-timeout

Conversation

@DanielTal87
Copy link
Contributor

Status

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

Related Issues

fix: https://jira-dc.paloaltonetworks.com/browse/XSUP-61776

Description

bug fix

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @content-bot start review - Initiate a full AI code review
  • @content-bot re-review - Incremental review for new commits

@content-bot
Copy link
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

github-actions bot commented Feb 4, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/ProofpointThreatResponse/Integrations/ProofpointThreatResponseEventCollector
   ProofpointThreatResponseEventCollector.py1362383%46, 48–49, 118–119, 121, 134–135, 137–139, 141, 161–163, 165, 177, 188, 193, 381–383, 449
TOTAL1362383% 

Tests Skipped Failures Errors Time
2 0 💤 0 ❌ 0 🔥 3.021s ⏱️

@DanielTal87 DanielTal87 changed the title ProofpointThreatResponse - Add diagnostic logging and optimize batch fetch loop [XSUP-61776] ProofpointThreatResponse - Add diagnostic logging and optimize batch fetch loop Feb 4, 2026
@content-bot
Copy link
Contributor

Validate summary
The following errors were reported as warnings: DO106.
The following errors were thrown as a part of this pr: PA114, RN106.
The following errors cannot be ignored: PA114, RN106.
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: PA114, RN106.

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.

2 participants