[superset] branch arash/logAlertReports created (now aa1741a23d)

[aa...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

aafghahi pushed a change to branch arash/logAlertReports
in repository https://gitbox.apache.org/repos/asf/superset.git


      at aa1741a23d additional logging

This branch includes the following new commits:

     new aa1741a23d additional logging

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[superset] 01/01: additional logging

[aa...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

aafghahi pushed a commit to branch arash/logAlertReports
in repository https://gitbox.apache.org/repos/asf/superset.git

commit aa1741a23d7cc5e6858e5cc7ee60868e74a1ca01
Author: AAfghahi <ar...@gmail.com>
AuthorDate: Thu Oct 13 11:41:21 2022 -0400

    additional logging
---
 superset/reports/notifications/email.py | 4 +++-
 superset/utils/core.py                  | 6 ++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/superset/reports/notifications/email.py b/superset/reports/notifications/email.py
index 358294f1e1..1f042ded83 100644
--- a/superset/reports/notifications/email.py
+++ b/superset/reports/notifications/email.py
@@ -207,6 +207,8 @@ class EmailNotification(BaseNotification):  # pylint: disable=too-few-public-met
                 dryrun=False,
                 header_data=content.header_data,
             )
-            logger.info("Report sent to email")
+            logger.info(
+                "Report sent to email, notification content is %s", content.header_data
+            )
         except Exception as ex:
             raise NotificationError(ex) from ex
diff --git a/superset/utils/core.py b/superset/utils/core.py
index 2786cfa27f..7fba35f09a 100644
--- a/superset/utils/core.py
+++ b/superset/utils/core.py
@@ -1003,8 +1003,9 @@ def send_mime_email(
     smpt_ssl_server_auth = config["SMTP_SSL_SERVER_AUTH"]
 
     if dryrun:
-        logger.info("Dryrun enabled, email notification content is below:")
-        logger.info(mime_msg.as_string())
+        logger.info(
+            "Dryrun enabled, email notification content is %s", mime_msg.as_string()
+        )
         return
 
     # Default ssl context is SERVER_AUTH using the default system
@@ -1020,6 +1021,7 @@ def send_mime_email(
     if smtp_user and smtp_password:
         smtp.login(smtp_user, smtp_password)
     logger.debug("Sent an email to %s", str(e_to))
+    logger.info("Sent email notification content is %s", mime_msg.as_string())
     smtp.sendmail(e_from, e_to, mime_msg.as_string())
     smtp.quit()