Skip to content

[AWSINTS-3658] feat(go-forwarder): add SecurityHub#1128

Merged
ndakkoune merged 8 commits into
nabil.dakkoune/go-forwarderfrom
nabil.dakkoune/AWSINTS-3658
May 20, 2026
Merged

[AWSINTS-3658] feat(go-forwarder): add SecurityHub#1128
ndakkoune merged 8 commits into
nabil.dakkoune/go-forwarderfrom
nabil.dakkoune/AWSINTS-3658

Conversation

@ndakkoune
Copy link
Copy Markdown
Contributor

@ndakkoune ndakkoune commented May 15, 2026

What does this PR do?

Add SecurityHub logs coming from EventBridge.

Motivation

Testing Guidelines

Additional Notes

  • Fixed WAF gunzip (it does not always write a gzip file, used the same logic as in Python by checking the magic headers).

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)
  • This PR passes the unit tests
  • This PR passes the installation tests (ask a Datadog member to run the tests)

@github-actions github-actions Bot added the aws label May 15, 2026
@ndakkoune ndakkoune marked this pull request as ready for review May 18, 2026 12:41
@ndakkoune ndakkoune requested a review from a team as a code owner May 18, 2026 12:41
@ndakkoune ndakkoune requested review from ViBiOh and ge0Aja May 19, 2026 08:59
Comment thread aws/logs_monitoring_go/internal/handling/cloudtrail.go
@ndakkoune ndakkoune merged commit 13ec167 into nabil.dakkoune/go-forwarder May 20, 2026
10 checks passed
@ndakkoune ndakkoune deleted the nabil.dakkoune/AWSINTS-3658 branch May 20, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants