-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
fetch all computers without a date filter. #37219
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great job! please see my comments
Packs/JamfProtect/Integrations/JamfProtectEventCollector/JamfProtectEventCollector.yml
Show resolved
Hide resolved
Packs/JamfProtect/Integrations/JamfProtectEventCollector/JamfProtectEventCollector_test.py
Show resolved
Hide resolved
Packs/JamfProtect/Integrations/JamfProtectEventCollector/JamfProtectEventCollector.py
Show resolved
Hide resolved
required: false | ||
advanced: true | ||
section: Collect | ||
additionalinfo: When set to true, retrieves all available computers during the initial fetch, ignoring the Events Fetch Interval and not limited to recent changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
additionalinfo: When set to true, retrieves all available computers during the initial fetch, ignoring the Events Fetch Interval and not limited to recent changes. | |
additionalinfo: When set to true, retrieves all available computers during the initial fetch. |
@@ -16,6 +16,13 @@ Use this integration to fetch audit logs, alerts and computer events from Jamf P | |||
| Max alert events per fetch | Maximum number of alert events to fetch at a time. Default is 1000| False | | |||
| Max audit events per fetch | Maximum number of audit events to fetch at a time. Default is 20,000| False | | |||
| Max computer events per fetch | Maximum number of computer events to fetch at a time. Default is 1000| False | | |||
| Fetch all computers | When set to true, retrieves all available computers during the initial fetch, ignoring the Events Fetch Interval and not limited to recent changes. | False | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Fetch all computers | When set to true, retrieves all available computers during the initial fetch, ignoring the Events Fetch Interval and not limited to recent changes. | False | | |
| Fetch all computers | When set to true, retrieves all available computers during the initial fetch. | False | |
|
||
|
||
|
||
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
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
Related Issues
fixes: CIAC-12027
Description
A few sentences describing the overall goals of the pull request's commits.
Must have