Skip to content

[aws] Rebrand AWS Security Hub to AWS Security Hub CSPM - #16195

Merged
brijesh-elastic merged 9 commits into
elastic:mainfrom
brijesh-elastic:aws-4.6.0
Dec 11, 2025
Merged

[aws] Rebrand AWS Security Hub to AWS Security Hub CSPM#16195
brijesh-elastic merged 9 commits into
elastic:mainfrom
brijesh-elastic:aws-4.6.0

Conversation

@brijesh-elastic

@brijesh-elastic brijesh-elastic commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Proposed commit message

aws: Rebrand AWS Security Hub to AWS Security Hub CSPM

The changes include display names, reference links, documentation and dashboards to align
with the new brand name.
This adds a breaking change, as it involves updating the `observer.vendor` value to
`AWS Security Hub CSPM`.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

Integration Overview

Before:
AWS Security Hub

After:
AWS Security Hub CSPM

@brijesh-elastic brijesh-elastic self-assigned this Dec 2, 2025
@brijesh-elastic
brijesh-elastic requested review from a team as code owners December 2, 2025 11:18
@brijesh-elastic brijesh-elastic added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Dec 2, 2025
@brijesh-elastic
brijesh-elastic requested a review from a team as a code owner December 2, 2025 11:18
@brijesh-elastic brijesh-elastic added enhancement New feature or request Integration:aws AWS dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Dec 2, 2025
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@jamiehynds

Copy link
Copy Markdown

@brijesh-elastic please don't merge this PR until there's approval from PM (@cpascale43). We need a better understand of the AWS branding and whether Security Hub CSPM is simply a component within Security Hub (which I think is the case). If we reband, it likely implies a narrower focus than what the integration offers.

@cpascale43 can you look into this one please and confirm if we should stick with the current Security Hub branding?

@brijesh-elastic
brijesh-elastic marked this pull request as draft December 2, 2025 11:28
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

Package aws 👍(9) 💚(6) 💔(7)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
lambda_logs 9803.92 6622.52 -3181.4 (-32.45%) 💔
route53_public_logs 20408.16 16666.67 -3741.49 (-18.33%) 💔
route53_resolver_logs 12987.01 6944.44 -6042.57 (-46.53%) 💔
securityhub_findings_full_posture 1968.5 1037.34 -931.16 (-47.3%) 💔
vpcflow 7936.51 6024.1 -1912.41 (-24.1%) 💔
cloudfront_logs 2898.55 2386.63 -511.92 (-17.66%) 💔
elb_logs 5235.6 3759.4 -1476.2 (-28.2%) 💔

To see the full report comment with /test benchmark fullreport

@cpascale43

Copy link
Copy Markdown
Contributor

We are good to merge this - it was discussed here

AWS has branded them as two new services - these can be configured separately so it makes sense to me to treat them as two separate integrations:
https://docs.aws.amazon.com/securityhub/latest/userguide/what-are-securityhub-services.html

However, since this is relatively new (~June 2025) we should be prepared for feedback

@efd6 efd6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit only

Comment thread packages/aws/changelog.yml Outdated
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
@brijesh-elastic
brijesh-elastic marked this pull request as ready for review December 3, 2025 05:29
Comment thread packages/aws/img/logo_securityhub.svg

@efd6 efd6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM after conflicts are addressed.

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@maxcold, since we are rebranding our existing integration AWS Security Hub to AWS Security Hub CSPM, there will be change required in this list and its corresponding page.

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Requires observer.vendor field to be updated to AWS Security Hub CSPM, confirmed by @nick-alayil.

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Insights also come under CSPM. Can you update the securityhub_insights data stream as well?

@brijesh-elastic

Copy link
Copy Markdown
Contributor Author

Insights also come under CSPM. Can you update the securityhub_insights data stream as well?

Updated in commit.

Comment thread packages/aws/changelog.yml Outdated
@@ -1,4 +1,11 @@
# newer versions go on top
- version: "4.8.0"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- version: "4.8.0"
- version: "5.0.0"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

for breaking-change

@brijesh-elastic brijesh-elastic added breaking change and removed enhancement New feature or request labels Dec 10, 2025

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @brijesh-elastic

@agithomas agithomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@brijesh-elastic
brijesh-elastic merged commit 8e2c355 into elastic:main Dec 11, 2025
8 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws - 5.0.0 containing this change is available at https://epr.elastic.co/package/aws/5.0.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:aws AWS Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

8 participants