Skip to content

Conversation

@netpok
Copy link
Contributor

@netpok netpok commented Nov 17, 2021

The current implementation tries to use startls instead of tls.

This is incorrect because

  • starttls: opens a tcp connection and later upgrades to tls
  • tls: opens the tls connection and does not need to upgrade later
@Webklex
Copy link
Owner

Webklex commented Nov 17, 2021

Hi @netpok ,
many thanks for this pull request! Great work :)

Best regards,

@Webklex Webklex merged commit 14a3432 into Webklex:master Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants