Skip to content

Commit

Permalink
docs(event-targets): clarify that adding an imported SNS Topic as an …
Browse files Browse the repository at this point in the history
…event target does not set required permissions (aws#30279)

### Issue aws#25583 

Closes aws#25583

### Reason for this change

Add documentation to clarify that imported topics have to have the required permissions set manually.

### Description of changes

Added docstring.

### Description of how you validated changes



### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
daschaa authored and atanaspam committed Jun 3, 2024
1 parent 75aa95d commit a18bad0
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions packages/aws-cdk-lib/aws-events-targets/lib/sns.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ export interface SnsTopicProps extends TargetBaseProps {

/**
* Use an SNS topic as a target for Amazon EventBridge rules.
* If the topic is imported the required permissions to publish to that topic need to be set manually.
*
* @example
* /// fixture=withRepoAndTopic
Expand Down

0 comments on commit a18bad0

Please sign in to comment.