[GitHub] [airflow] Nazarii commented on issue #19594: EmailOperator is not compatible with the SendGrid backend when hosted on MWAA.

[GitBox <gi...@apache.org>]

Nazarii commented on issue #19594:
URL: https://github.com/apache/airflow/issues/19594#issuecomment-969007146


   @uranusjr I don't think this is smth to fixed on the MWAA, sengrid provider should follow the similar flow defined in `EmailOperator`:
   
   1. When sending emails with `EmailOperator` we do not specify the sender email address. This address will be taken from the `AIRFLOW__SMTP__SMTP_MAIL_FROM` setting.
   2. Sending an email with the SendGrid provider as a backend requires setting up the `SENDGRID_MAIL_FROM`. It will be good if we have a single entry point configuration for different mail systems, so we do not set up the sender email per provider. What I propose is to use `AIRFLOW__SMTP__SMTP_MAIL_FROM` option if `SENDGRID_MAIL_FROM` is not set.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org