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

optional STARTTLS support #494

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

optional STARTTLS support #494

wants to merge 1 commit into from

Conversation

akostadinov
Copy link

@akostadinov akostadinov commented Jan 4, 2022

fixes #142

You can test by:

openssl req -new -newkey rsa:4096 -days 365 -nodes -x509 -subj "/C=US/ST=Denial/L=Springfield/O=Dis/CN=localhost" -keyout certs/localhost.key -out certs/localhost.cert

bundle exec mailcatcher --smtp-starttls --smtp-private-key certs/localhost.key --smtp-cert certs/localhost.cert

openssl s_client -starttls smtp -connect localhost:1025

AUTH PLAIN XDB0ZXN0X3VzZXJAd29zaHViLmNvbVwwKW5n

Auth is just a bonus tip actually. By default eventmachine smtpserver accepts any username/password pair in PLAIN auth.

@onpaws
Copy link

onpaws commented May 4, 2022

@akostadinov Thanks for your efforts here!

Is there a Docker image anywhere I could try?
I'm curious to try this out but don't have Ruby on this machine at the moment.

@akostadinov
Copy link
Author

I don't have. You need to either install ruby or enter a shell of a suitable existing container image.

@daniloargentiero
Copy link

Please merge this,
thank you.

@robbl-as
Copy link

@akostadinov I think this PR has a higher chance to be merged, when documentation is added. Do you mind adding the new options to the README.md?

@akostadinov
Copy link
Author

@robbl-as , I don't really have time and I'm not very good at writing docs so it costs me a lot of stress. If you like, you can submit a PR against my branch and I will merge it. Or paste a patch as a comment.

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.

SSL/TLS connection
4 participants