Repository navigation
[v3-4-test] Add emailer utilities to the SMTP provider (#74623) - #75704
Merged
Merged
Conversation
* Add core email utilities to the SMTP provider Copy airflow.utils.email into airflow.providers.smtp.utils.emailer so that core can later deprecate its email module and delegate to the provider, once a provider release containing it is available. The module is a copy of the core module. The only code changes are the imports, which go through common.compat, and the Connection lookup, which uses BaseHook.get_connection as other providers do instead of importing airflow.models. SMTP settings still come from the [smtp] configuration section and only the login and password from the Connection. Routing these calls through SmtpHook instead would change behavior for existing setups, because the hook reads all settings from the Connection and retries differently. Tests are copied from core, with additional edge cases. The module is registered as an email backend. * Describe the SMTP emailer without referring to core The emailer documentation now explains what the module does and how it is configured, without mentioning where the code came from. (cherry picked from commit 3199497) Co-authored-by: Shivam Rastogi <6463385+shivaam@users.noreply.github.com>
1 task done
vatsrahul1001
marked this pull request as ready for review
October 27, 2026 03:41
vatsrahul1001
enabled auto-merge (squash)
October 27, 2026 03:41
Contributor
Author
|
Hi maintainer, this PR was merged without a milestone set.
|
vatsrahul1001
added a commit
that referenced
this pull request
Oct 27, 2026
airflow-core now requires providers-smtp>=3.2.0 (email deprecation, #75414/#75709). The emailer source landed via #75704 but the in-repo provider version was still 3.1.1, so the CI image build failed resolving the floor. Mirrors the smtp slice of the 3.2.0 provider release (#75209): version + provider.yaml + changelog + doc refs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Copy airflow.utils.email into airflow.providers.smtp.utils.emailer so that
core can later deprecate its email module and delegate to the provider,
once a provider release containing it is available.
The module is a copy of the core module. The only code changes are the
imports, which go through common.compat, and the Connection lookup, which
uses BaseHook.get_connection as other providers do instead of importing
airflow.models. SMTP settings still come from the [smtp] configuration
section and only the login and password from the Connection. Routing these
calls through SmtpHook instead would change behavior for existing setups,
because the hook reads all settings from the Connection and retries
differently.
Tests are copied from core, with additional edge cases. The module is
registered as an email backend.
The emailer documentation now explains what the module does and how it is
configured, without mentioning where the code came from.
(cherry picked from commit 3199497)
Co-authored-by: Shivam Rastogi 6463385+shivaam@users.noreply.github.com