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

Fix armis get events command #30877

Merged
merged 21 commits into from Nov 24, 2023
Merged

Fix armis get events command #30877

merged 21 commits into from Nov 24, 2023

Conversation

thefrieddan1
Copy link
Contributor

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.

Description

Fixes the command armis-get-events, add to it an event type argument

@thefrieddan1 thefrieddan1 self-assigned this Nov 14, 2023
Copy link
Contributor

@DeanArbel DeanArbel left a comment

Choose a reason for hiding this comment

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

No comment, very well done.

Packs/Armis/Integrations/ArmisEventCollector/README.md Outdated Show resolved Hide resolved
Packs/Armis/ReleaseNotes/1_1_8.md Outdated Show resolved Hide resolved
Packs/Armis/ReleaseNotes/1_1_8.md Outdated Show resolved Hide resolved
@thefrieddan1 thefrieddan1 merged commit 159567f into master Nov 24, 2023
17 checks passed
@thefrieddan1 thefrieddan1 deleted the fix_armis_get_events_command branch November 24, 2023 05:22
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* Fixing armis-get-events

* Adjust unit test to changes.

* handle armis-get-events if no events return

* restore-alerts-parsing-rule

* update-release-notes

* update-release-notes

* Fix eventType is missing error

* Fix unit tests.
Add AQL arg

* bump docker version.
update RN
update README

* Add unitest increase coverage

* Apply suggestions from code review

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* pre-commit failures

---------

Co-authored-by: Chanan Welt <cwelt@paloaltonetworks.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants