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

Xdr ioc not pushed #30039

Merged
merged 16 commits into from Oct 10, 2023
Merged

Xdr ioc not pushed #30039

merged 16 commits into from Oct 10, 2023

Conversation

daryakoval
Copy link
Contributor

Status

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

Related Issues

fixes: https://jira-hq.paloaltonetworks.local/browse/XSUP-28613

Description

Added validation to tell the customer that validation failed and some iocs not pushed.

Must have

  • Tests
  • Documentation

@daryakoval daryakoval marked this pull request as ready for review October 4, 2023 11:58
Copy link
Contributor

@dansterenson dansterenson left a comment

Choose a reason for hiding this comment

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

Looks good!

dorschw
dorschw previously requested changes Oct 4, 2023
#### Integrations
##### Cortex XDR - IOC

- Added a validation for pushed indicators, using ***xdr-ioc-push*** command.
Copy link
Contributor

@dorschw dorschw Oct 4, 2023

Choose a reason for hiding this comment

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

Packs/CortexXDR/ReleaseNotes/5_1_9.md Outdated Show resolved Hide resolved
Packs/CortexXDR/Integrations/XDR_iocs/XDR_iocs.py Outdated Show resolved Hide resolved
return {'data': json.dumps({"request_data": _json})}
data = {"request_data": _json}
if validate:
data['validate'] = True
Copy link
Contributor

Choose a reason for hiding this comment

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

This flag basically tells the API to return validation error info?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes

Packs/CortexXDR/Integrations/XDR_iocs/XDR_iocs.py Outdated Show resolved Hide resolved
Packs/CortexXDR/Integrations/XDR_iocs/XDR_iocs_test.py Outdated Show resolved Hide resolved
daryakoval and others added 5 commits October 8, 2023 18:10
Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@users.noreply.github.com>
Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@users.noreply.github.com>
@daryakoval daryakoval merged commit d3fddbd into master Oct 10, 2023
17 of 18 checks passed
@daryakoval daryakoval deleted the xdr-ioc-not-pushed branch October 10, 2023 09:31
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* save adding validation

* save rn

* fix test

* fix ruff

* changed to warning

* changed to warning

* save rn

* Update Packs/CortexXDR/ReleaseNotes/5_1_9.md

Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@users.noreply.github.com>

* update docstring

---------

Co-authored-by: Shahaf Ben Yakir <44666568+ShahafBenYakir@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