diff --git a/changelogs/fragments/tls_mail.yml b/changelogs/fragments/tls_mail.yml new file mode 100644 index 00000000000..2858897ba35 --- /dev/null +++ b/changelogs/fragments/tls_mail.yml @@ -0,0 +1,2 @@ +bugfixes: + - fix mail notification module when using starttls and py3.7 diff --git a/lib/ansible/modules/notification/mail.py b/lib/ansible/modules/notification/mail.py index c5fe5867275..7e4ff356346 100644 --- a/lib/ansible/modules/notification/mail.py +++ b/lib/ansible/modules/notification/mail.py @@ -248,8 +248,8 @@ def main(): try: if secure != 'never': try: - smtp = smtplib.SMTP_SSL(host=host, timeout=timeout) - code, smtpmessage = smtp.connect(host, port=port) + smtp = smtplib.SMTP_SSL(host=host, port=port, timeout=timeout) + code, smtpmessage = smtp.connect(host, port) secure_state = True except ssl.SSLError as e: if secure == 'always': @@ -259,8 +259,8 @@ def main(): pass if not secure_state: - smtp = smtplib.SMTP(timeout=timeout) - code, smtpmessage = smtp.connect(host, port=port) + smtp = smtplib.SMTP(host=host, port=port, timeout=timeout) + code, smtpmessage = smtp.connect(host, port) except smtplib.SMTPException as e: module.fail_json(rc=1, msg='Unable to Connect %s:%s: %s' % (host, port, to_native(e)), exception=traceback.format_exc())