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

use version independent TLS method because TLSv1_2_method is deprecated #19

Closed
wants to merge 3 commits into from

Conversation

ursfassler
Copy link

We can not build the code on an up to date Debian system because the function is deprecated and -Werror enabled.

@chaurah
Copy link
Contributor

chaurah commented Jun 20, 2017

Hi @ursfassler,
Thank you for this submission. We are going to rework the OpenSSL wrapper in the next release to add support for v1.1 of OpenSSL. This change will be included alongside.
Please do let us know if you have further suggestions.

Rahul

@chaurah
Copy link
Contributor

chaurah commented Oct 12, 2017

The OpenSSL related changes have been merged to master. We will have another commit with the rest of the changes from this pull request before closing this.

Rahul

@vareddy-zz
Copy link
Contributor

vareddy-zz commented Oct 13, 2017

Closing this as the rest of the changes are a duplicate of #29

@vareddy-zz vareddy-zz closed this Oct 13, 2017
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.

4 participants