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