Skip to content

[CFT] Add ElasticAwsSecurityHub policy to federated-identity-aws template - #8030

Merged
seanrathier merged 2 commits into
mainfrom
seanrathier/federated-identity-aws-remaining
Aug 20, 2026
Merged

[CFT] Add ElasticAwsSecurityHub policy to federated-identity-aws template#8030
seanrathier merged 2 commits into
mainfrom
seanrathier/federated-identity-aws-remaining

Conversation

@seanrathier

@seanrathier seanrathier commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the ElasticAwsSecurityHub IAM policy resource to deploy/cloudformation/federated-identity-aws.yml, granting the permissions required by the aws_securityhub and aws package SecurityHub data streams for Federated Identity (Cloud Connectors) deployments.

The remaining packages in elastic/ingest-dev#8812 (aws_bedrock, aws_bedrock_agentcore, aws_mq, aws_logs) require no new policy blocks — their streams read exclusively from CloudWatch metrics and CloudWatch Logs, which are already covered by the existing ElasticAwsMetrics and ElasticAwsCloudwatchLogs policies.

Permissions added

Action Authorizes
securityhub:GetFindings POST /findings (aws/securityhub_findings, aws/securityhub_findings_full_posture) and POST /findingsv2 (aws_securityhub/finding) — both operations use this IAM action
securityhub:GetInsights POST /insights/get (aws/securityhub_insights)

Related

🤖 Generated with Claude Code

@mergify

mergify Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @seanrathier? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@mergify

mergify Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b seanrathier/federated-identity-aws-remaining upstream/seanrathier/federated-identity-aws-remaining
git merge upstream/main
git push upstream seanrathier/federated-identity-aws-remaining

…late

Grants the IAM permissions required by the aws/securityhub and aws_securityhub
integrations for Federated Identity (Cloud Connectors) deployments.

securityhub:GetFindings authorizes both POST /findings and POST /findingsv2 —
both SecurityHub API operations share this single IAM action.
securityhub:GetInsights authorizes POST /insights/get.

Part of elastic/ingest-dev#8812. Paired with elastic/integrations#20529.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mergify

mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

The aws-cloudwatch input requires DescribeLogStreams, GetLogEvents, and
GetLogGroupFields in addition to the existing DescribeLogGroups and
FilterLogEvents. These are declared in provider_permissions across all
federated aws-cloudwatch packages (aws_mq, aws_bedrock, aws_bedrock_agentcore,
aws_logs, kubernetes); the CFT must grant them.

Added:
  - logs:DescribeLogStreams
  - logs:GetLogEvents
  - logs:GetLogGroupFields

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@seanrathier
seanrathier enabled auto-merge August 20, 2026 13:22
@seanrathier
seanrathier added this pull request to the merge queue Aug 20, 2026
Merged via the queue into main with commit ba7c269 Aug 20, 2026
13 checks passed
@seanrathier
seanrathier deleted the seanrathier/federated-identity-aws-remaining branch August 20, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip Team:Security-Cloud Services Security Data Experience - Cloud Services team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants