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

Add an option for mandatory TLS on SMTPServer #107

Open
danerubado-wk opened this issue May 8, 2024 · 0 comments
Open

Add an option for mandatory TLS on SMTPServer #107

danerubado-wk opened this issue May 8, 2024 · 0 comments

Comments

@danerubado-wk
Copy link

danerubado-wk commented May 8, 2024

Hello!

I recently attempted to switch to this library to replace https://github.com/go-gomail/gomail as it has been abandoned. I specifically needed the custom connection feature that is provided here to customize the dialer.

Additionally, I wanted to be able to specify that TLS is mandatory.

Right now the behavior for Encryption.STARTTLS is such that it checks with the server to see if STARTTLS is supported. If it is not, it simply falls back to not using TLS. I appreciate this because I can "opportunistically" enable STARTTLS in my application without having to worry about the server.

Still, I would like the option to throw an error if the SMTPServer is configured with TLSMandatory.

gomail had the same issue and merged a fix for it described here: go-mail/mail#23

I tried to put up a PR to this effect but I do not have write access to the repo. Is this something we think we could support?

@danerubado-wk danerubado-wk changed the title Add an option for mandatory TLS Add an option for mandatory TLS on SMTPServer May 8, 2024
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

No branches or pull requests

1 participant