[GitHub] [airflow] dbarrundiag opened a new issue #21671: Amazon Airflow Provider | Broken AWS SES as backend for Email

[GitBox]

dbarrundiag opened a new issue #21671:
URL: https://github.com/apache/airflow/issues/21671


   ### Apache Airflow Provider(s)
   
   amazon
   
   ### Versions of Apache Airflow Providers
   
   ```
   apache-airflow==2.2.2
   apache-airflow-providers-amazon==2.4.0
   ```
   
   ### Apache Airflow version
   
   2.2.2
   
   ### Operating System
   
   Amazon Linux 2
   
   ### Deployment
   
   MWAA
   
   ### Deployment details
   
   _No response_
   
   ### What happened
   
   As part of this PR https://github.com/apache/airflow/pull/18042 the 
signature of the function 
`airflow.providers.amazon.aws.utils.emailer.send_email` is no longer compatible 
with how `airflow.utils.email.send_email` invokes the function. Essentially the 
functionally of using SES as Email Backend is broken.
   
   ### What you expected to happen
   
   This behavior is erroneous because the signature of 
`airflow.providers.amazon.aws.utils.emailer.send_email` should be compatible 
with how we call the backend function in `airflow.utils.email.send_email`:
   
   ```
       return backend(
           to_comma_separated,
           subject,
           html_content,
           files=files,
           dryrun=dryrun,
           cc=cc,
           bcc=bcc,
           mime_subtype=mime_subtype,
           mime_charset=mime_charset,
           conn_id=backend_conn_id,
           **kwargs,
       )
   ```
   
   ### How to reproduce
   
   ## Use AWS SES as Email Backend
   [email]
   email_backend = airflow.providers.amazon.aws.utils.emailer.send_email
   email_conn_id = aws_default
   
   ## Try sending an Email
   ```
   from airflow.utils.email import send_email
   
   def email_callback(**kwargs):
       send_email(to=['t...@hello.io'], subject='test', html_content='content')
   
   email_task = PythonOperator(
       task_id='email_task',
       python_callable=email_callback,
   )
   ```
   
   ## The bug shows up
   ```
     File "/usr/local/airflow/dags/environment_check.py", line 46, in 
email_callback
       send_email(to=['t...@hello.io'], subject='test', html_content='content')
     File "/usr/local/lib/python3.7/site-packages/airflow/utils/email.py", line 
66, in send_email
       **kwargs,
   TypeError: send_email() missing 1 required positional argument: 
'html_content'
   ```
   
   ### Anything else
   
   Every time.
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscr...@airflow.apache.org

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