Skip to content

Reintroduce Amazon web services module#8065

Merged
Desvelao merged 16 commits intomainfrom
enhancement/8055-reintroduce-amazon-web-services-module
Feb 13, 2026
Merged

Reintroduce Amazon web services module#8065
Desvelao merged 16 commits intomainfrom
enhancement/8055-reintroduce-amazon-web-services-module

Conversation

@gonzaarancibia
Copy link
Member

@gonzaarancibia gonzaarancibia commented Jan 22, 2026

Description

Reintroduce a functional Amazon web services module adapting the visualizations to the available fields in Wazuh 5.0.0

Issues Resolved

#8055

Evidence

Screenshot

Screenshot 2026-01-22 at 5 27 52 PM Screenshot 2026-01-22 at 5 56 05 PM

Test

  • Verify that the aws section render correctly

Check List

  • All tests pass
    • yarn test:jest
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@gonzaarancibia gonzaarancibia linked an issue Jan 22, 2026 that may be closed by this pull request
@gonzaarancibia gonzaarancibia marked this pull request as ready for review January 22, 2026 20:56
gonzaarancibia and others added 3 commits January 22, 2026 17:56
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
@rodrigofez
Copy link
Contributor

🟢 LGTM

Test Chrome Firefox Safari
Verify that the aws section render correctly 🟢 🟢
Chrome

Screenshot 2026-01-22 at 18 01 44 Screenshot 2026-01-22 at 18 02 16

Safari

Screenshot 2026-01-22 at 18 07 06 Screenshot 2026-01-22 at 18 07 22

Copy link
Contributor

@rodrigofez rodrigofez left a comment

Choose a reason for hiding this comment

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

LGTM

It only looks like you need to run prettier in plugins/main/public/utils/applications.ts

@rodrigofez rodrigofez self-assigned this Feb 6, 2026
Ripdiegozz
Ripdiegozz previously approved these changes Feb 6, 2026
Copy link
Contributor

@Ripdiegozz Ripdiegozz left a comment

Choose a reason for hiding this comment

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

CR 🟢
Tests 🟢

Image Image

Copy link
Member

@FernandoCastilla FernandoCastilla left a comment

Choose a reason for hiding this comment

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

LGTM

Image

@Adman23 Adman23 self-requested a review February 11, 2026 12:37
Copy link
Contributor

@Ripdiegozz Ripdiegozz left a comment

Choose a reason for hiding this comment

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

🔴 Tests

  1. When sorting the aws.cloudtrail_event_source field in the events table, it throws the following error.
2026-02-11.10-47-12.mp4

Adman23

This comment was marked as outdated.

@Ripdiegozz Ripdiegozz self-assigned this Feb 12, 2026
@Desvelao
Copy link
Member

Desvelao commented Feb 12, 2026

Test

Image Image

🔴 Tests

  1. When sorting the aws.cloudtrail_event_source field in the events table, it throws the following error.

2026-02-11.10-47-12.mp4

Sorting works for me, maybe this could be fixed in a recent version of indexer or you need to update the resource files of indexer.

Copy link
Member

@Desvelao Desvelao left a comment

Choose a reason for hiding this comment

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

LGTM!

@Desvelao Desvelao requested a review from Ripdiegozz February 12, 2026 17:36
@Desvelao
Copy link
Member

I just replicated the commented problem by @Ripdiegozz with other table columns:

image

@Ripdiegozz Ripdiegozz removed their request for review February 12, 2026 20:38
@github-actions
Copy link
Contributor

Wazuh Check Updates plugin code coverage (Jest) test % values
Statements 78.72% ( 185 / 235 )
Branches 62.72% ( 69 / 110 )
Functions 61.7% ( 29 / 47 )
Lines 78.72% ( 185 / 235 )

@github-actions
Copy link
Contributor

Wazuh Core plugin code coverage (Jest) test % values
Statements 29.82% ( 190 / 637 )
Branches 14.78% ( 42 / 284 )
Functions 19.65% ( 34 / 173 )
Lines 30% ( 189 / 630 )

@Desvelao
Copy link
Member

Desvelao commented Feb 13, 2026

I just replicated the commented problem by @Ripdiegozz with other table columns:

image

I just started a new environment and this seems to work again the first times I tried.

Yesterday we saw if changing the order of the columns in the sort, the problem could not appear or filtering by the document where the aws.cloudtrail_event_source field exist.

I think we could continue with the revision and merging the PR due to the error is not consistent, this could be reviewed in a future issue if this happens again.

EDIT: after a time with the environment, I experienced the problem.

Response of the request ``` { "error": { "root_cause": [], "type": "search_phase_execution_exception", "reason": "", "phase": "fetch", "grouped": true, "failed_shards": [], "caused_by": { "type": "unsupported_operation_exception", "reason": null } } } ```

The error happens on Discover app if using the Time and the aws.cloudtrail_event_source field as sorting order.
image

@github-actions
Copy link
Contributor

Main plugin code coverage (Jest) test % values
Statements 20.42% ( 5496 / 26914 )
Branches 15.65% ( 2597 / 16590 )
Functions 17.51% ( 1174 / 6704 )
Lines 20.66% ( 5347 / 25878 )

@Desvelao Desvelao dismissed Ripdiegozz’s stale review February 13, 2026 08:16

We will not block the pull request due to the error found related to sorting, this seems that happens sometimes.

@Desvelao Desvelao merged commit 770aec3 into main Feb 13, 2026
7 of 9 checks passed
@Desvelao Desvelao deleted the enhancement/8055-reintroduce-amazon-web-services-module branch February 13, 2026 08:16
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.

Reintroduce Amazon web services module

7 participants