Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DKIM implementations #5620

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

DorianCoding
Copy link

Hello,

I would like to implement DKIM on Rust emails, so SPF and DKIM can be fulfilled and allows better transmission of mails.

Thanks.

@tessus
Copy link
Contributor

tessus commented Feb 20, 2025

May I ask why you want to implement this on the client? By client I mean the mail client. vw server acts as a mail client (unless the sendmail binary is directly used).

DKIM and SPF are usually handled by the mail server. Also, what happens if you use DKIM on the client and then on the server again? Or actually it's worse, when using DKIM on the client but not on the server. The server will add headers that in turn will render the DKIM signature created on the client invalid.
Have you tested all possible combinations?

@BlackDex
Copy link
Collaborator

@tessus, sendmail is nothing more then a client in our case. It's how the sendmail is configured which might make it differently

@DorianCoding
Copy link
Author

DorianCoding commented Feb 20, 2025

Hello,

Yes I know but some person might use DKIM on their own without their server and it would be wise to allow people to do it if they want to, because the implementation is minimal as you can see. It might not be useful for everyone, I fully agree and if the server already manages SPF/DKIM and so on, there is no need to implement anything more but it might not be the case and people should be able to implement it. I've not tested all combinations but rapid tests shows it can work if people are actually configuring it themselves before and wants to continue.

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.

4 participants