[aws] Enable Identity Federation for SecurityHub data streams - #20529
Draft
seanrathier wants to merge 3 commits into
Draft
[aws] Enable Identity Federation for SecurityHub data streams#20529seanrathier wants to merge 3 commits into
seanrathier wants to merge 3 commits into
Conversation
…-dev#8812) Migrates securityhub_findings, securityhub_findings_full_posture, and securityhub_insights HTTPJSON templates from hand-rolled SigV4 signing to the input's auth.aws configuration. Removes the identity_federation gate from the securityhub policy template. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
9 tasks
Contributor
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
✅ All changelog entries have the correct PR link. |
💚 Build Succeeded
History
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Important
Parked. The
awspackage's securityhub template has an open deprecate-or-coexist question versus the standaloneaws_securityhubpackage, which is the intended vehicle for Security Hub federation (#20436, paired with elastic/cloudbeat#7588). This PR stays draft until that decision lands; if the legacy template is deprecated, close this unmerged.Proposed commit message
Enable Identity Federation for the
awspackage SecurityHub data streams.WHAT: migrates
securityhub_findings,securityhub_findings_full_posture, andsecurityhub_insightsHTTPJSON templates from hand-rolled SigV4 signing (X-Amz-Date header + Authorization set transforms, including the pagination re-sign) to the input'sauth.awsconfiguration withuse_cloud_connectorsdriven by thesupports_identity_federationvar. Removes theidentity_federationgate from the securityhub policy template.WHY: the hand-rolled signing never set
X-Amz-Security-Token, so session tokens were silently broken, and it blocked the Cloud Connectors zero-credential flow. Part of ingest-dev#8812 (SecurityHub federation is scoped there, not under ingest-dev#8802).Checklist
changelog.ymlfile.Author's Checklist
AccessDeniedException, streamsHEALTHY)securityhub:GetInsightsstill needs adding on the cloudbeat side — Add aws_securityhub permissions to the Federated Identity template cloudbeat#7588 currently carriessecurityhub:GetFindingsonlyHow to test this PR locally
For the federated path E2E:
securityhub:GetFindingsandsecurityhub:GetInsightson the rolesecurityhub_findings,securityhub_findings_full_posture, andsecurityhub_insightsall showHEALTHYwith noAccessDeniedExceptionin agent logsRelated issues
GetInsightsadded)awspackage federation, under elastic/ingest-dev#8802)