feat(events-targets): warn when using SQS with imported KMS keys

[maczac150]

Issue # (if applicable)

Closes #35628

Reason for this change

When using EventBridge rules to send events to SQS queues encrypted with imported KMS keys, CDK cannot automatically configure the required KMS key policy permissions. This is because imported keys (via Key.fromKeyArn() or Key.fromLookup()) have an undefined policy that CDK cannot modify.

Without proper KMS permissions, EventBridge fails to send messages to the encrypted queue at runtime, but CDK provides no indication of this misconfiguration during synthesis or deployment.

Root Cause Analysis

When queue.grantSendMessages() is called with a ServicePrincipal (EventBridge), CDK's grant mechanism attempts to add permissions in two ways:

• Principal Policy: ServicePrincipal is a non-identity principal and does not have its own policy document. The addToPrincipalPolicy() method in principals.ts always returns {statementAdded: false} for non-identity principals.

• Resource Policy: For imported keys (fromKeyArn() or fromLookup()), the policy property is undefined. When addToResourcePolicy() is called in key.ts, it returns {statementAdded: false} because there is no policy to modify.

This dual failure occurs regardless of whether the KMS key and EventBridge are in the same account or different accounts, because:

• Same account: Grant.addToPrincipalOrResource() tries principal first, then resource policy

• Cross-account: Grant.addToPrincipalAndResource() tries both principal and resource policies

In both code paths no statements are added, which leaves the stack misconfigured without any indication at synthesis or deployment time.

Detection Logic

The fix uses Resource.isOwnedResource() to distinguish between CDK-managed and imported keys:

Key Type	isOwnedResource	Behavior
new Key(...)	true	No warning
Key.fromCfnKey(cfnKey)	true	No warning
Key.fromKeyArn(...)	false	Emits warning
Key.fromLookup(...)	false	Emits warning

Description of changes

• Added validateKmsKeyPermissions() method in SqsQueue target to detect imported KMS keys using Resource.isOwnedResource()

• Emit a warning when an imported KMS key is detected, instructing users to manually add required permissions (kms:Decrypt and kms:GenerateDataKey) to the key policy

• Added comprehensive documentation in README with usage examples and manual configuration instructions

Describe any new or updated permissions being added

None.
This change only adds a synthesis-time warning and does not modify any IAM permissions. The warning reminds users to manually configure KMS key policy permissions (kms:Decrypt, kms:GenerateDataKey) for imported keys, which CDK cannot modify automatically.

Description of how you validated changes

• Unit tests: Added new tests in sqs.test.ts to verify that a warning is emitted only for imported KMS keys and not for CDK-managed or unencrypted queues.
• Integration test: Created integ.sqs-imported-key.ts to demonstrate the warning scenario with an imported KMS key, verifying that the message includes the key ID and required permissions (kms:Decrypt, kms:GenerateDataKey).

Checklist

• My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

---

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

[aws-cdk-automation]

This PR has been in the MERGE CONFLICTS state for 3 weeks, and looks abandoned. Note that PRs with failing linting check or builds are not reviewed, please ensure your build is passing

To prevent automatic closure:

• Resume work on the PR
• OR request an exemption by adding a comment containing 'Exemption Request' with justification e.x "Exemption Request: "
• OR request clarification by adding a comment containing 'Clarification Request' with a question e.x "Clarification Request: "

This PR will automatically close in 14 days if no action is taken.

[aws-cdk-automation]

This PR has been deemed to be abandoned, and will be automatically closed. Please create a new PR for these changes if you think this decision has been made in error.

[github-actions]

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.