Skip to content
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

Bug fix/XSUP-43864/FireEye NX fixed fetch incidents #37263

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

Conversation

MosheEichler
Copy link
Contributor

Status

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

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-43864

Description

Updated the start time (used in the fetch-incidents command) to be later than 48 hours ago.

Must have

  • Tests
  • Documentation

@MosheEichler MosheEichler self-assigned this Nov 18, 2024
@MosheEichler MosheEichler added the release-notes-only Indicates that this pull request has ONLY release notes to review for documentation process label Nov 18, 2024
Copy link

github-actions bot commented Nov 18, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/FireEyeNX/Integrations/FireEyeNX
   FireEyeNX.py5401397%219, 437, 540, 580–581, 583–584, 834, 844, 871, 883, 1383, 1498
TOTAL5401397% 

Tests Skipped Failures Errors Time
189 0 💤 0 ❌ 0 🔥 28.855s ⏱️

@ShirleyDenkberg
Copy link
Contributor

@thefrieddan1 Doc review completed.

…x-fetch-incidnts

# Conflicts:
#	Packs/FireEyeNX/Integrations/FireEyeNX/FireEyeNX.yml
#	Packs/FireEyeNX/ReleaseNotes/1_1_34.md
@MosheEichler MosheEichler marked this pull request as ready for review November 27, 2024 12:59
@@ -25,6 +25,7 @@

DEFAULT_SESSION_TIMEOUT = 15 * 60 # In Seconds
DEFAULT_FETCH_LIMIT = '50'
TIME_BACK_IN_SECONDS = 172000.0 # Due to API limitations we can retrieve events only from the last 48 hours.
Copy link
Contributor

Choose a reason for hiding this comment

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

change constant name to FORTY_EIGHT_HOURS_IN_SECOND
or keep the name and do TIME_BACK_IN_SECONDS = 60 * 60 * 48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approved release-notes-only Indicates that this pull request has ONLY release notes to review for documentation process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants