Feature request: SES configuration set delivery options

[logsnitch]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Hi, I'd like to request a new resource for creating Amazon SES Configuration Sets "DeliveryOptions": https://docs.aws.amazon.com/ses/latest/APIReference/API_DeliveryOptions.html

New or Affected Resource(s)

aws_ses_configuration_set

Potential Terraform Configuration

resource "aws_ses_configuration_set" "enforce-tls" {
  name = "enforce-tls"

  delivery_options {
    tls_policy = "Require"
  }
}

References

• https://docs.aws.amazon.com/ses/latest/APIReference/API_DeliveryOptions.html

[marcjay]

Hi, is this on any roadmap to address or would a PR for this feature be welcomed? This is a feature we'd like to see implemented as well

[marcjay]

I've raised #11600 based on the syntax above. Happy to change it to support a different syntax if there's a preference.

[marcjay]

Hi @bflad please could I get your view on the delivery_options syntax here: #11197 (comment) ? If you would like it to be something different I'd be happy to update the PR. Many thanks

[adamdecaf]

I added this as a new resource which also has sending_pool for #10703. If #11600 is merged I can add sending_pool on the existing aws_ses_configuration_set resource for my change.

My Change: adamdecaf@16045d0

[l13t]

Hi there. It would be amazing to have possibility to configure TLS policy. We're now configuring SES in our infra and need to have a hack to set this option.

[adamdecaf]

I never got a reply on this repo for the original PR's, but my fork has been used in production. I'd dust off the PR's if there's a chance they'll get merged.

[marcjay]

Hi there. It would be amazing to have possibility to configure TLS policy. We're now configuring SES in our infra and need to have a hack to set this option.

@l13t if you add a 👍 to #11600, that I think is the only way to encourage the PR to get merged, I've given up trying to nudge people

[l13t]

@marcjay already did it and asked colleagues :)

[ghost]

This has been released in version 3.28.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!