Skip to content

feat: Improved SMTP error responses#147

Merged
j-g00da merged 1 commit into
mainfrom
j-g00da/better-smtp-errors
May 12, 2026
Merged

feat: Improved SMTP error responses#147
j-g00da merged 1 commit into
mainfrom
j-g00da/better-smtp-errors

Conversation

@j-g00da
Copy link
Copy Markdown
Collaborator

@j-g00da j-g00da commented May 7, 2026

Fixes: #140

@j-g00da j-g00da requested a review from link2xt May 7, 2026 11:41
Comment thread src/error.rs
@j-g00da j-g00da force-pushed the j-g00da/better-smtp-errors branch 2 times, most recently from 07e586a to 6f6e65d Compare May 7, 2026 13:21
Comment thread src/smtp_client.rs Outdated
@j-g00da j-g00da force-pushed the j-g00da/better-smtp-errors branch from 6f6e65d to 5e783a0 Compare May 12, 2026 06:08
@j-g00da j-g00da requested a review from hpk42 May 12, 2026 06:10
Copy link
Copy Markdown
Contributor

@hpk42 hpk42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, but indeed makes sense to avoid reporting back internal details to users.

Comment thread src/error.rs
@j-g00da j-g00da force-pushed the j-g00da/better-smtp-errors branch 3 times, most recently from 163781c to 3b0608a Compare May 12, 2026 10:20
Fixes: #140
Signed-off-by: Jagoda Ślązak <jslazak@jslazak.com>
@j-g00da j-g00da force-pushed the j-g00da/better-smtp-errors branch from 3b0608a to b9f6a34 Compare May 12, 2026 10:20
@j-g00da j-g00da merged commit 39b97d3 into main May 12, 2026
7 checks passed
@j-g00da j-g00da deleted the j-g00da/better-smtp-errors branch May 12, 2026 10:37
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.

Include mx_host into the error message

2 participants