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.

Sign up for GitHub

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

Jump to bottom

Rework of the MITRE alerts view #1313

Merged
merged 6 commits into from
May 5, 2022
Merged

Rework of the MITRE alerts view #1313

merged 6 commits into from
May 5, 2022

Conversation

AlexRuiz7
Copy link
Member

@AlexRuiz7 AlexRuiz7 commented Apr 29, 2022 •
edited
Loading

Summary

Closes #1234

Due to the extension of the MITRE Framework on Wazuh, this section had to be reworked in order to provide the new functionalities.

mitre-splunk

To test

  • Go to the MITRE ATT&CK module on the Overview section.
  • Click on Show alerts
  • Check that the alerts correspond to the alerts shown on the Dashboard.
  • Check that clicking on each MITRE Technique opens a model with information about it.
  • Check that no error happens.
  • Check that information shown makes sense.
  • Note that some MITRE Technique details are not provided yet by the API, so N/A or No description provided. is shown instead.

@AlexRuiz7 AlexRuiz7 self-assigned this Apr 29, 2022
yenienserrano
yenienserrano approved these changes May 2, 2022
View reviewed changes
Copy link
Member

@yenienserrano yenienserrano left a comment

Choose a reason for hiding this comment

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

Test: ✔️
CR: ✔️

@chantal-kelm chantal-kelm self-requested a review May 5, 2022 11:45
chantal-kelm
chantal-kelm approved these changes May 5, 2022
View reviewed changes
Copy link
Member

@chantal-kelm chantal-kelm left a comment

Choose a reason for hiding this comment

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

CR: ✔️
Test: ✔️

@Machi3mfl Machi3mfl changed the base branch from master to 4.3 May 5, 2022 12:28
@matiasmoreno876 matiasmoreno876 merged commit cd675e1 into 4.3 May 5, 2022
@AlexRuiz7 AlexRuiz7 deleted the fix/1234-mitre-overview-alerts branch May 16, 2022 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yenienserrano yenienserrano yenienserrano approved these changes

@chantal-kelm chantal-kelm chantal-kelm approved these changes

Assignees

@AlexRuiz7 AlexRuiz7

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The controller overviewMitreIds uses null and undefined class attributes
5 participants
@AlexRuiz7 @yenienserrano @chantal-kelm @matiasmoreno876 @asteriscos