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 AWSRecreateSG EC2 breaking change bug #32962

Merged
merged 8 commits into from Feb 26, 2024
Merged

Conversation

jlevypaloalto
Copy link
Contributor

Status

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

Related Issues

fixes: link to the issue

Description

AWSRecreateSG fails after the recent changes to AWS - EC2 (version 1.3.0)
The problems are caused by the change in the object returned when AWS - EC2 commands are called using the demisto.executeCommand function in the script . This PR uses a new path to extract the required data correctly.

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Feb 15, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AWS-Enrichment-Remediation/Scripts/AWSRecreateSG
   AWSRecreateSG.py1894874%35–39, 50, 89, 94–98, 100–101, 109–110, 112, 132, 143–144, 146, 151–153, 164–165, 167, 172–177, 179–180, 202, 204, 208, 235, 281, 316, 323–324, 336–340
TOTAL1894874% 

Tests Skipped Failures Errors Time
9 0 💤 0 ❌ 0 🔥 2.042s ⏱️

Copy link
Contributor

@johnnywilkes johnnywilkes left a comment

Choose a reason for hiding this comment

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

tested in XSIAM and XSOAR. This makes the script no longer backwards compatible, but I don't see this as a concern

Copy link
Contributor

@johnnywilkes johnnywilkes left a comment

Choose a reason for hiding this comment

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

please re-add this block

@jlevypaloalto jlevypaloalto merged commit 3ce4932 into master Feb 26, 2024
21 checks passed
@jlevypaloalto jlevypaloalto deleted the jl-AWSRecreateSG-fix branch February 26, 2024 14:10
rundssoar pushed a commit to rundssoar/content that referenced this pull request Feb 28, 2024
* fixed

* remove print

* update docker + RN

* add BC warning

* remove duplicate

* fixed unit-tests

* newline
maimorag pushed a commit that referenced this pull request Feb 28, 2024
* fixed

* remove print

* update docker + RN

* add BC warning

* remove duplicate

* fixed unit-tests

* newline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants