Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(ses): add configurationSetArn property on ConfigurationSet #29902

Open
1 of 2 tasks
fredericbarthelet opened this issue Apr 19, 2024 · 2 comments
Open
1 of 2 tasks

(ses): add configurationSetArn property on ConfigurationSet #29902

fredericbarthelet opened this issue Apr 19, 2024 · 2 comments
Labels
@aws-cdk/aws-ses Related to Amazon Simple Email Service effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p3

Comments

@fredericbarthelet
Copy link
Contributor

fredericbarthelet commented Apr 19, 2024

Describe the feature

Add a configurationSetArn property exposed on ConfigurationSet construct to improve building policies for resource using this ConfigurationSet to send emails with SES.

Use Case

When granting send email access to a lambda the ARN for the ConfigurationSet must be constructed manually:

Grant.addToPrincipal({
    grantee,
    actions: ["ses:SendEmail"],
    resourceArns: [
      this.stack.formatArn({
        service: 'ses',
        resource: 'configuration-set',
        resourceName: 'marketing-team-configuration-set',
      }),
    ],
    scope: this
  })

Proposed Solution

Similar to addition made in #29084 , proposed solution includes building with formatArn method the resource underlying ARN.

Other Information

This paves the way for future grant methods on ConfigurationSet construct

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.137.0

Environment details (OS name and version, etc.)

macOS 14.2.1

@fredericbarthelet fredericbarthelet added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Apr 19, 2024
@github-actions github-actions bot added the @aws-cdk/aws-ses Related to Amazon Simple Email Service label Apr 19, 2024
@khushail khushail added investigating This issue is being investigated and/or work is in progress to resolve the issue. and removed needs-triage This issue or PR still needs to be triaged. labels Apr 19, 2024
@khushail
Copy link
Contributor

thanks for requesting and submitting the PR @fredericbarthelet

@khushail khushail added p2 effort/small Small work item – less than a day of effort and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Apr 19, 2024
@pahud pahud added p3 and removed p2 labels Jun 11, 2024
@aprat84
Copy link

aprat84 commented Dec 10, 2024

Exposing configuration set ARN is a must.

Maybe the grant methods on the Identity could also grant to the configuration set, if present?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
@aws-cdk/aws-ses Related to Amazon Simple Email Service effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants