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

Taxii2 fix #27348

Merged
merged 3 commits into from
Jun 14, 2023
Merged

Taxii2 fix #27348

merged 3 commits into from
Jun 14, 2023

Conversation

YuvHayun
Copy link
Contributor

@YuvHayun YuvHayun commented Jun 11, 2023

Status

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

Related Issues

fixes: CIAC-6920

Description

  • Fixed an issue where the same extension was shown to all objects.

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

@YuvHayun YuvHayun requested a review from JudahSchwartz June 11, 2023 10:13
Copy link
Contributor

@JudahSchwartz JudahSchwartz left a comment

Choose a reason for hiding this comment

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

good catch

@@ -934,7 +931,7 @@ def create_extension_definition(object_type, extensions_dict, xsoar_type,
'version': '1.0',
'extension_types': ['property-extension']
}
extensions_dict[object_type] = {'extension_id': extension_id, 'xsoar_indicator_to_return': xsoar_indicator_to_return}
extensions_dict[object_type] = True
Copy link
Contributor

Choose a reason for hiding this comment

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

whats this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since I don't really need to keep track of the values, this is enough to assign a boolean value to flag that his object_type already has an extension definition.

@YuvHayun YuvHayun requested a review from JudahSchwartz June 13, 2023 12:19
@YuvHayun YuvHayun merged commit 48f4de1 into master Jun 14, 2023
@YuvHayun YuvHayun deleted the taxii2_fix branch June 14, 2023 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants