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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to