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

Allow raw tls_options to be passed through to gen_smtp #223

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arjan
Copy link

@arjan arjan commented Aug 31, 2023

Since OTP 26, TLS checking has changed a bit. For gen_smtp it is now necessary (see this ticket) to pass in the tls_options which are retrieved from the :tls_certificate_check erlang module:

 tls_options: :tls_certificate_check.options(relay)

This patch allows the :tls_options configuration key to be passed through as-is to gen_smtp.

@nclark
Copy link

nclark commented Feb 22, 2024

Is there a current recommendation for users of bamboo_smtp 4.2.2? I'm troubleshooting an application that stopped sending mail after upgrading to OTP26 with the {:error, {:temporary_failure, ~c"52.192.253.213", :tls_failed}} type of error. It seems like the only thing that works is tls_verify: :verify_none which is not ideal.

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