mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 08:15:52 +03:00
[PR #610] [CLOSED] enhancement: Emails do not have a message_id #850
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#850
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/lldap/lldap/pull/610
Author: @SamantaTarun
Created: 6/22/2023
Status: ❌ Closed
Base:
main← Head:tarun_samanta/#608📝 Commits (3)
836626denhancement: Emails do not have a message_idb5e746atrying to fix error3903769fixed some error📊 Changes
1 file changed (+1 additions, -0 deletions)
View changed files
📝
server/src/infra/mail.rs(+1 -0)📄 Description
Closes #608
Issue Description:
I noticed that password-reset-mails sent from my system have a higher spam score as expected.
This is because the mails are missing the message-id header.
Usually it is up to the originating client to add this header.
But many email-providers add it automatically if it is missing, because those mails would very likely get flagged as spam and lower their trust score.
Anyway it would be a good thing to not rely on the mailserver to "repair" the mails for us.
I can not implement it myself but here is how another project which is also using lettre solved it.
Maybe someone can get a PR going.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.