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

Misp v3 delete attribute + local tag arg #17742

Merged
merged 9 commits into from
Feb 20, 2022
Merged

Conversation

abaumgarten
Copy link
Contributor

@abaumgarten abaumgarten commented Feb 17, 2022

Status

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

Related Issues

fixes: https://github.com/demisto/etc/issues/46774

Description

  • Added new command misp-delete-attribute.
  • In the misp-add-tag-to-event command the is_local argument was added.
  • In the misp-add-tag-to-attribute command the is_local argument was added.

Screenshots

*1
image
*2
image
*3
image

Minimum version of Cortex XSOAR

  • 5.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests - TBP was updated.
  • Documentation - readme was updated.

@bakatzir FYI

@abaumgarten abaumgarten self-assigned this Feb 17, 2022
@@ -2,7 +2,7 @@
"name": "MISP",
"description": "Malware information and threat sharing platform.",
"support": "xsoar",
"currentVersion": "2.0.6",
"currentVersion": "2.0.7",
Copy link
Member

Choose a reason for hiding this comment

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

Should bump a minor as I see it https://semver.org/.

@@ -0,0 +1,5 @@
#### Integrations
##### MISP v3
- Added new command ***misp-delete-attribute***.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Added new command ***misp-delete-attribute***.
- Added the command ***misp-delete-attribute*** which allow users to delete an attribute according to the given attribute ID.

Copy link
Contributor

@guykeller guykeller left a comment

Choose a reason for hiding this comment

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

Please just fix the documentation (per my note), and consider if UTs should be added even though the TPB was enhanced

Comment on lines 3 to 5
- Added the command ***misp-delete-attribute*** which allow users to delete an attribute according to the given attribute ID.
- In the **misp-add-tag-to-event** command the *is_local* argument was added.
- In the **misp-add-tag-to-attribute** command the *is_local* argument was added.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Added the command ***misp-delete-attribute*** which allow users to delete an attribute according to the given attribute ID.
- In the **misp-add-tag-to-event** command the *is_local* argument was added.
- In the **misp-add-tag-to-attribute** command the *is_local* argument was added.
- Added the command ***misp-delete-attribute*** which allows users to delete an attribute according to a given attribute ID.
- Added the *is_local* argument to the **misp-add-tag-to-event** command.
- Added the *is_local* argument to the **misp-add-tag-to-attribute** command.

@content-bot
Copy link
Collaborator

@content-bot
Copy link
Collaborator

The following integrations/tests were collected by the CI build but are currently skipped. The collected tests are related to this pull request and might be critical.:

  • Impossible Traveler - Test - reason: Usage limit reached (Issue 38063)

@abaumgarten abaumgarten merged commit ab8bf67 into master Feb 20, 2022
@abaumgarten abaumgarten deleted the misp_v3_delete_attribute branch February 20, 2022 17:38
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.

4 participants