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

Error connecting smtp with STARTTLS #19

Open
thiagoalmeidasa opened this issue Apr 25, 2017 · 4 comments
Open

Error connecting smtp with STARTTLS #19

thiagoalmeidasa opened this issue Apr 25, 2017 · 4 comments

Comments

@thiagoalmeidasa
Copy link
Contributor

I'm getting

ERROR: Error: 140500104592424:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:s23_clnt.c:794

when try to connect my smtp server using STARTTLS.

I solved my problem following this instructions
nodemailer/nodemailer#358 (comment)

We need to add the requireTLS option from nodeMailer in this image. I can use the PR #9 as a model and write my own PR, but I would like to receive your comments before I begin to write.

killianbrackey pushed a commit that referenced this issue Apr 25, 2017
@killianbrackey
Copy link
Contributor

Do the changes in the above PR cover some of the fixes you mentioned here and in #18 @thiagoalmeidasa ?

@thiagoalmeidasa
Copy link
Contributor Author

Yes, it worked. Thanks 😄

killianbrackey pushed a commit that referenced this issue Apr 25, 2017
@killianbrackey
Copy link
Contributor

Awesome! I will close the issues after this get merged in. Let me know if anything else comes up!

@thiagoalmeidasa
Copy link
Contributor Author

So, I did a research in nodemailer examples and found this:
https://github.com/nodemailer/nodemailer/blob/5be7c902e953043be3618d747c349f02405f003b/lib/well-known/services.json#L88

Works by adding the TLS ciphers like in the example.

secureConnection: false,
 tls: {
  ciphers: 'SSLv3'
  },

I don't know which solution is better in this case, but with both instructions, the SMTP with STARTTLS works fine.
Let me know what do you think.

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

2 participants