Skip to content

fix(deps): bump github.com/wneessen/go-mail from 0.7.1 to 0.7.2#3372

Merged
HFO4 merged 1 commit into
cloudreve:masterfrom
WittF:fix/upgrade-go-mail-v0.7.2
Mar 26, 2026
Merged

fix(deps): bump github.com/wneessen/go-mail from 0.7.1 to 0.7.2#3372
HFO4 merged 1 commit into
cloudreve:masterfrom
WittF:fix/upgrade-go-mail-v0.7.2

Conversation

@WittF

@WittF WittF commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Upgrade go-mail from v0.7.1 to v0.7.2 to fix a regression that strips the display name from the From header.

Problem

When sending emails via Tencent Cloud SES, the following error occurs:

内部错误 (Failed to send test email: send failed: closing SMTP DATA writer:
554 5.0.0 Error: transaction failed. Detailed error:
##SES-response-json:{"Response":{"RequestId":"5abbd1c2-8092-401d-9980-7c8988a9b42e",
"QcloudError":{"Error":{"Code":"FailedOperation.NotAuthenticatedSender",
"Message":"操作失败。发件sender没有经过认证,无法发送。"}}}}

Tencent Cloud SES requires the From header to include a display name (e.g. "WittF-Mail" <noreply@wit.tf>). A bare address like <noreply@wit.tf> is rejected.

Root Cause

go-mail v0.7.1 introduced a regression (wneessen/go-mail#497): GetSender(false) mutates the original *mail.Address pointer by setting addr.Name = "", which strips the display name from the From header before the message body is written over SMTP.

This was fixed in v0.7.2 (wneessen/go-mail#498) by using a value copy instead of a pointer copy.

Changes

  • go.mod — bump github.com/wneessen/go-mail v0.7.1 → v0.7.2
  • go.sum — updated checksums

Fix From header display name being stripped, which causes Tencent Cloud
SES to reject emails with "NotAuthenticatedSender".
@HFO4

HFO4 commented Mar 26, 2026

Copy link
Copy Markdown
Member

Thanks!

@HFO4 HFO4 merged commit 58b83ea into cloudreve:master Mar 26, 2026
1 of 2 checks passed
@WittF WittF deleted the fix/upgrade-go-mail-v0.7.2 branch June 1, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants