Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Amazon SES: add webhook extension points; close webhook boto3 clients #403

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

medmunds
Copy link
Contributor

@medmunds medmunds commented Sep 9, 2024

In Amazon SES webhook views (tracking and inbound):

  • Close boto3 clients after use. (Not strictly required, but doesn't hurt. Amazon SES backend was already doing this.)
  • Break out some webhook functionality to simplify subclassing. (E.g., to handle S3 object encryption through outside tooling, as AWS hasn't released a Python version of their S3 encryption client.)

In Amazon SES webhook views (tracking and inbound):
- Close boto3 clients after use. (Not strictly required, but doesn't hurt.
  Amazon SES backend was already doing this.)
- Break out some webhook functionality to simplify subclassing.
  (E.g., to handle S3 object encryption through outside tooling, as
  AWS hasn't released a Python version of their S3 encryption client.)
@medmunds medmunds merged commit 063fb08 into main Sep 9, 2024
49 checks passed
@medmunds medmunds deleted the feature/amazon-ses-webhook-extensions branch September 9, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant