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

integration fixes and improvements #26271

Merged
merged 2 commits into from May 3, 2023

Conversation

content-bot
Copy link
Collaborator

Original External PR

external pull request

Contributor

@rhallick

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

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

Related Issues

fixes: link to the issue

Description

Recent changes to the XSOAR platform have created issues for the integrations within the Intel 471 Content Pack. These changes resolve those issues, along with some other performance improvements.

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

* integration fixes and improvements

* changed content pack version get

* add rn and bump metadata

* add rn

* updated release notes

* added unit test

* Update Packs/FeedIntel471/Integrations/Intel471MalwareIndicator/Intel471MalwareIndicator.py

Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>

* update test_fetch_indicators test

* Update Packs/FeedIntel471/Integrations/Intel471MalwareIndicator/Intel471MalwareIndicator.py

Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>

* Update Packs/FeedIntel471/Integrations/Intel471MalwareIndicator/Intel471MalwareIndicator.yml

Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>

* Update Packs/FeedIntel471/ReleaseNotes/2_0_18.md

Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>

* Update Packs/FeedIntel471/pack_metadata.json

Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>

* change release number

* revert to original get method

* test_module and get_indicators_command fixes

* pan suggested changes

* removed obsolete code

* removed whitespace

* cr fixes + test fixes

---------

Co-authored-by: GuyAfik <guyafik11@gmail.com>
Co-authored-by: Guy Afik <53861351+GuyAfik@users.noreply.github.com>
@content-bot content-bot added Contribution Thank you! Contributions are always welcome! docs-approved Partner ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. Contribution Form Filled Whether contribution form filled or not. labels May 3, 2023
@content-bot content-bot requested a review from GuyAfik May 3, 2023 11:32
@GuyAfik GuyAfik merged commit 78c5b02 into master May 3, 2023
13 of 14 checks passed
@GuyAfik GuyAfik deleted the contrib/rhallick_integration-improvements branch May 3, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! docs-approved Partner ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR.
Projects
None yet
4 participants