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 iocs integration- 'XSOAR Comment Field Exporting To XDR' parameter is handled incorrectly #36658

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

rshunim
Copy link
Contributor

@rshunim rshunim commented Oct 8, 2024

Status

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

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-11945

Description

Due to a change in the parameter type (from type 0 - text, to type 16 - list), we encounter an error when updates are sent from XDR to XSOAR.
This error occurs because a list is being used as a key in a dictionary.
Handling it in this fix.

Must have

  • Tests
  • Documentation

@rshunim rshunim requested a review from dorschw October 8, 2024 13:23
@rshunim rshunim self-assigned this Oct 8, 2024
@rshunim rshunim requested a review from maimorag as a code owner October 8, 2024 13:23
Copy link

github-actions bot commented Oct 8, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CortexXDR/Integrations/XDR_iocs
   XDR_iocs.py5197585%47, 102, 113, 157, 214, 244, 247–251, 260–261, 401, 488–489, 495–498, 500, 535–540, 542–544, 566–569, 571, 595–597, 599–600, 631, 645, 650, 665, 697, 710–712, 721–726, 730–733, 735–739, 742–746, 763, 782–785, 813, 819
TOTAL5197585% 

Tests Skipped Failures Errors Time
405 0 💤 0 ❌ 0 🔥 9.699s ⏱️

Copy link
Contributor

@dorschw dorschw left a comment

Choose a reason for hiding this comment

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

DM

@rshunim rshunim requested a review from tcarmeli1 October 9, 2024 12:48
@rshunim
Copy link
Contributor Author

rshunim commented Oct 13, 2024

The fix was checked with @tcarmeli1 and seems to resolve the issue.

@rshunim
Copy link
Contributor Author

rshunim commented Oct 13, 2024

I'll fix the UTs according to the fix and wait with merging the PR since there is an issue with mirroring the 'comments' from XDR to XSOAR

@dorschw FYI

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • CortexXDR pack version was bumped to 6.1.84.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • CortexXDR pack version was bumped to 6.1.85.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@rshunim rshunim requested a review from dorschw October 30, 2024 12:35
@rshunim
Copy link
Contributor Author

rshunim commented Oct 30, 2024

@tcarmeli1 @dorschw please review it both of you

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • CortexXDR pack version was bumped to 6.1.86.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants