You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tooling.apache.org by wa...@apache.org on 2026/04/14 19:11:26 UTC

(tooling-trusted-releases) 01/01: Do not log the email message body

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

wave pushed a commit to branch email-logging-content
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git

commit 327d519002246bb611a0f4e0e35efcce8d63eed1
Author: Dave Fisher <da...@comcast.net>
AuthorDate: Tue Apr 14 12:11:14 2026 -0700

    Do not log the email message body
---
 atr/mail.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/atr/mail.py b/atr/mail.py
index 4c8d1134..a2ba0d68 100644
--- a/atr/mail.py
+++ b/atr/mail.py
@@ -57,7 +57,9 @@ class Message:
 
 async def send(msg_data: Message, category: MailFooterCategory) -> tuple[str, list[str]]:
     """Send an email notification about an artifact or a vote."""
-    log.info(f"Sending email for event: {msg_data}")
+    log.info(
+        "Sending email for event", sender=msg_data.email_sender, recipient=msg_data.email_to, subject=msg_data.subject
+    )
     _reject_null_bytes(
         msg_data.email_sender,
         msg_data.email_to,
@@ -108,7 +110,6 @@ async def send(msg_data: Message, category: MailFooterCategory) -> tuple[str, li
     start = time.perf_counter()
     # Convert to string to satisfy the existing _send_many function signature
     msg_text = msg.as_string()
-    log.info(f"sending message: {msg_text}")
 
     errors = await _send_many(from_addr, all_envelope_recipients, msg_text)
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@tooling.apache.org
For additional commands, e-mail: commits-help@tooling.apache.org