Skip to content

fix(event-targets): update dlq policy to handle encrypted messages - #38706

Open
duranbe wants to merge 12 commits into
aws:mainfrom
duranbe:fix-dlq-permissions
Open

fix(event-targets): update dlq policy to handle encrypted messages#38706
duranbe wants to merge 12 commits into
aws:mainfrom
duranbe:fix-dlq-permissions

Conversation

@duranbe

@duranbe duranbe commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Issue #33169

Closes #33169

Reason for this change

Fix the DLQ policy for encrypted Event Bridge Target

Describe any new or updated permissions being added

  • Granting Decrypt and GenerateDataKey to the DLQ

Description of how you validated changes

Unit Test Added

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@github-actions github-actions Bot added effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2 labels Aug 30, 2026
@aws-cdk-automation
aws-cdk-automation requested a review from a team August 30, 2026 16:04
@github-actions github-actions Bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Aug 30, 2026

@aws-cdk-automation aws-cdk-automation left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

(This review is outdated)

@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Aug 30, 2026
@aws-cdk-automation
aws-cdk-automation dismissed their stale review September 1, 2026 16:52

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

aws-cdk-lib/aws-events-targets: SqsQueue target not granting access to the KMS key

2 participants